Discussion related to cado-nfs

[Alexander Kruppa <alexander.kruppa@inria.fr>]

----- Original Message -----
> From: "Christophe Clavier" <christophe.clavier@unilim.fr>
> To: Cado-nfs-discuss@lists.gforge.inria.fr
> Sent: Sunday, 24 November, 2013 2:27:00 AM
> Subject: Re: [Cado-nfs-discuss] CADO-NFS 2.0: Small issue with factor.sh
> 
> 
> Sorry, I meant that the fix was :
> 
>       $hostnames cores=$cores
> 
> Regards
> 
> Christophe


Hi Christophe,

thanks for the report! This is indeed a bug, introduced in commit d3d6a82c. I 
added this fix:

 if [ -z "$CADO_USEHOST" ] && echo "$hostnames" | grep -E "^(localhost, 
*)*localhost$" > /dev/null
 then
-   server_address="server.address=localhost"
+   host=localhost # For the Perl script
+   server_address="server.address=localhost" # For the Python script
 else
+   host=`hostname --short`
    server_address=""
 fi

Note that the $hostnames variable is really meant for the Python script, 
i.e., it is set by the -h parameter. It defaults to localhost, so without an 
-h parameter (which should not be used with -pl), it'll happen to work, but 
CADO_USEHOST would not be taken into account.

Alex


 
> Le 23/11/2013 18:05, Christophe Clavier a écrit :
> >
> > Hello,
> >
> > I wonder whether I have found a small bug in the factor.sh script
> > of
> > CADO-NFS 2.0.
> >
> > Line 287 reads as:
> >      $host cores=$cores
> > but the variable host is not defined.
> >
> > I locally fixed the problem by modifying this line as:
> >      $host cores=$cores
> > which writes "localhost cores=xxx" (instead of "   cores=xxx") in
> > the
> > machine description file, but I am not sure this is the correct way
> > to
> > fix the problem.
> >
> > Regards
> >
> > Christophe
> >
> >
> > _______________________________________________
> > Cado-nfs-discuss mailing list
> > Cado-nfs-discuss@lists.gforge.inria.fr
> > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/cado-nfs-discuss
> 
> 
> _______________________________________________
> Cado-nfs-discuss mailing list
> Cado-nfs-discuss@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/cado-nfs-discuss
>