Nut upsdev - Dec 2010 - [nut-commits] svn commit r2748 - trunk

Citeren Arnaud Quette <aquette op alioth.debian.org>:
> Author: aquette
> Date: Fri Dec 17 23:32:20 2010
> New Revision: 2748
> URL: http://trac.networkupstools.org/projects/nut/changeset/2748
>
> Log:
> --with-doc=auto now behave depending on AsciiDoc presence
Instead of building whatever can be build (depending on the versions  
found), you now have changed the behavior to an 'all-or-nothing'  
approach. I'm not sure that I like this.

Best regards, Arjen
-- 
Please keep list traffic on the list (off-list replies will be rejected)

2010/12/18 Arjen de Korte
> Citeren Arnaud Quette
>
>
>  Author: aquette
>> Date: Fri Dec 17 23:32:20 2010
>> New Revision: 2748
>> URL: http://trac.networkupstools.org/projects/nut/changeset/2748
>>
>> Log:
>> --with-doc=auto now behave depending on AsciiDoc presence
>>
>
> Instead of building whatever can be build (depending on the versions
> found), you now have changed the behavior to an 'all-or-nothing'
approach.
> I'm not sure that I like this.
>
well, I'm not sure that I like that too.

it was designed to fix the Fedora buildbot slave, which failed.
but the issue is that since we don't version manpages, BB slaves must have
AsciiDoc to generate manpage.
so I'll revert this commit.

@Chetan: can you please check to install AsciiDoc 8.6.3?

cheers,
Arnaud
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.alioth.debian.org/pipermail/nut-upsdev/attachments/20101218/dde64013/attachment.htm>

Hi Charles,
> From: Charles Lepple [mailto:clepple at gmail.com]
> Sent: Monday, December 20, 2010 5:46 PM
> 
> On Dec 20, 2010, at 12:33 AM, <ChetanAgarwal at Eaton.com>
> <ChetanAgarwal at Eaton.com
>  > wrote:
> 
> > I have installed AsciiDoc 8.6.3 on the Fedora box. I checked the
> > compile by performing make distcheck-light. It was successful.
> 
> If it is in /usr/local, please add that to $PATH:
> 
> http://buildbot.networkupstools.org/public/nut/builders/Fedora-
> x64/builds/233/steps/configure/logs/stdio
> 
> environment:
> ...
>    PATH=/sbin:/usr/sbin:/bin:/usr/bin
> ...
> checking for asciidoc... /usr/bin/asciidoc
> checking for asciiDoc version... 8.2.5 found
> 
I have modified the PATH variable. Can you force the build to check if
it works?

Below is "./configure" output for reference - 

....
checking for asciidoc... (cached) /usr/local/bin/asciidoc
checking for a2x... (cached) /usr/local/bin/a2x
....
build and install documentation: yes
only build specific documentation format:  html-single html-chunked
build and install the development files: no
....


Thanks & Regards,
Chetan Agarwal
http://www.eaton.com/

> -----Original Message-----
> From: Charles Lepple [mailto:clepple at gmail.com]
> Sent: Tuesday, December 21, 2010 5:26 AM
> 
> Hi Chetan,
> 
> I don't know how Fedora sets $PATH for services like BuildBot, but the
> change doesn't seem to have worked:
> 
> http://buildbot.networkupstools.org/public/nut/builders/Fedora-
> x64/builds/238/steps/cleanup/logs/stdio
> 
> Is it being reset in an init script?
> 
Hello Charles,

Thanks to Arnaud, $PATH was augmented by adding "/usr/local/bin" path
in
"/etc/rc.local" init script file, which launches the Buildbot slave on
Fedora. 

I have tried forcing the build to check the compile and found issue
resolved:

http://buildbot.networkupstools.org/public/nut/builders/Fedora-x64/build
s/240/steps/compile/logs/stdio


Best Regards,
Chetan