[Nix-dev] Broken package

[Pjotr Prins]

I found the patch - sitting in a git tree somewhere. Not sure it
is needed any longer as the package compiles fine now. Anyway, still
kinda strange the build farm depends on packages that are (1) non
critical and unused, as this one was new and (2) not included in the
daily build list.

The explanation I thank you for, as it was missing in your SVN 
commit. However, I think the SVN tree *is* a testing tree or
development tree. We should get to a point where the build farm
manages a stable tree of sorts. Nix is a moving target now, which is a
questionable approach in the medium term. 

Pj.

diff -r 2dab102b8f1c make.inc.example
--- a/make.inc.example  Mon Sep 22 17:31:51 2008 +0200
+++ b/make.inc.example  Mon Sep 22 17:56:57 2008 +0200
@@ -18,21 +18,21 @@ PLAT = _LINUX
 #  selected.  Define LOADER and LOADOPTS to refer to the loader and 
 #  desired load options for your machine.
 #
-FORTRAN  = g77 
+FORTRAN  = gfortran
 OPTS     = -funroll-all-loops -O3
 DRVOPTS  = $(OPTS)
 NOOPT    =
-LOADER   = g77
+LOADER   = gfortran
 LOADOPTS =
 #
 # Timer for the SECOND and DSECND routines
 #
 # Default : SECOND and DSECND will use a call to the EXTERNAL FUNCTION ETIME
-TIMER    = EXT_ETIME
+# TIMER    = EXT_ETIME
 # For RS6K : SECOND and DSECND will use a call to the EXTERNAL FUNCTION ETIME_
 # TIMER    = EXT_ETIME_
 # For gfortran compiler: SECOND and DSECND will use a call to the INTERNAL FUNCTION ETIME
-# TIMER    = INT_ETIME 
+TIMER    = INT_ETIME 
 # If your Fortran compiler does not provide etime (like Nag Fortran Compiler, etc...)
 # SECOND and DSECND will use a call to the INTERNAL FUNCTION CPU_TIME 
 # TIMER    = INT_CPU_TIME


On Sun, Oct 05, 2008 at 09:51:10PM +0400, Michael Raskin wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Pjotr Prins wrote:
> > Michael: a broken package is no reason to comment it out. Because of
> > late binding nothing will happen. I think the only acceptable
> 
> Unfortunately, it actually caused a few failures on buildfarm. Buildfarm
> really checks that all packages can be instantiated, so missing patch
> ruins everything.