Re: [Scons-dev] should src/engine/SCons/Tool/docbook/util/* be included in the packages?

[Bill Deegan]

More data:
Here's the failure output when running against package:

warning: failed to load external entity
"/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
cannot parse
/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl
TypeError: __str__ returned non-string (type NoneType):
  File
"/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
line 1361:
_exec_main(parser, values)
  File
"/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
line 1324:
_main(parser)
  File
"/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
line 1103:
nodes = _build_targets(fs, options, targets, target_top)
  File
"/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
line 1298:
jobs.run(postfunc = jobs_postfunc)
  File
"/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Job.py",
line 111:
self.job.start()
  File
"/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Job.py",
line 214:
task.failed()
  File
"/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
line 274:
sys.stderr.write(errfmt % (nodename, buildError))


On Mon, Sep 18, 2017 at 3:36 PM, Bill Deegan 
wrote:

> Dirk,
>
> Looks like it's not currently and when I run "python runtest.py -p
> src-tar-gz -a" to run against the generated packages there are many docbook
> related failures.
>
> Thoughts?
>
> _Bill
>
___
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Re: [Scons-dev] should src/engine/SCons/Tool/docbook/util/* be included in the packages?

[Paweł Tomulik]

The problem still persists:

I/O warning : failed to load external entity
"/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
scons: *** [build/doc/user/manual.fo] parserError : xmlParseFile() failed
scons: building terminated because of errors.

should I file an issue on GitHub?

W dniu 18.09.2017 o 21:45, Bill Deegan pisze:
> More data: 
> Here's the failure output when running against package:
> 
> warning: failed to load external entity
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
> cannot parse
> /home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl
> TypeError: __str__ returned non-string (type NoneType):
>   File
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> line 1361:
>     _exec_main(parser, values)
>   File
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> line 1324:
>     _main(parser)
>   File
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> line 1103:
>     nodes = _build_targets(fs, options, targets, target_top)
>   File
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> line 1298:
>     jobs.run(postfunc = jobs_postfunc)
>   File
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Job.py",
> line 111:
>     self.job.start()
>   File
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Job.py",
> line 214:
>     task.failed()
>   File
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> line 274:
>     sys.stderr.write(errfmt % (nodename, buildError))
> 
> 
> On Mon, Sep 18, 2017 at 3:36 PM, Bill Deegan  > wrote:
> 
> Dirk,
> 
> Looks like it's not currently and when I run "python runtest.py -p
> src-tar-gz -a" to run against the generated packages there are many
> docbook related failures.
> 
> Thoughts?
> 
> _Bill
> 
> 
> 
> 
> ___
> Scons-dev mailing list
> Scons-dev@scons.org
> https://pairlist2.pair.net/mailman/listinfo/scons-dev
> 


-- 
Pawel Tomulik
___
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Re: [Scons-dev] should src/engine/SCons/Tool/docbook/util/* be included in the packages?

[Bill Deegan]

Strange that it's referring to
"/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl",  it should be
referring to build/... for the unpacked version of the package.

Have you run python bootstrap.py first?

On Fri, Oct 5, 2018 at 1:35 PM Paweł Tomulik 
wrote:

> The problem still persists:
>
> I/O warning : failed to load external entity
> "/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
> scons: *** [build/doc/user/manual.fo] parserError : xmlParseFile() failed
> scons: building terminated because of errors.
>
> should I file an issue on GitHub?
>
> W dniu 18.09.2017 o 21:45, Bill Deegan pisze:
> > More data:
> > Here's the failure output when running against package:
> >
> > warning: failed to load external entity
> >
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
> > cannot parse
> >
> /home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl
> > TypeError: __str__ returned non-string (type NoneType):
> >   File
> >
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 1361:
> > _exec_main(parser, values)
> >   File
> >
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 1324:
> > _main(parser)
> >   File
> >
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 1103:
> > nodes = _build_targets(fs, options, targets, target_top)
> >   File
> >
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 1298:
> > jobs.run(postfunc = jobs_postfunc)
> >   File
> >
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Job.py",
> > line 111:
> > self.job.start()
> >   File
> >
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Job.py",
> > line 214:
> > task.failed()
> >   File
> >
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 274:
> > sys.stderr.write(errfmt % (nodename, buildError))
> >
> >
> > On Mon, Sep 18, 2017 at 3:36 PM, Bill Deegan  > > wrote:
> >
> > Dirk,
> >
> > Looks like it's not currently and when I run "python runtest.py -p
> > src-tar-gz -a" to run against the generated packages there are many
> > docbook related failures.
> >
> > Thoughts?
> >
> > _Bill
> >
> >
> >
> >
> > ___
> > Scons-dev mailing list
> > Scons-dev@scons.org
> > https://pairlist2.pair.net/mailman/listinfo/scons-dev
> >
>
>
> --
> Pawel Tomulik
> ___
> Scons-dev mailing list
> Scons-dev@scons.org
> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>
___
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Re: [Scons-dev] should src/engine/SCons/Tool/docbook/util/* be included in the packages?

[Dirk Bächle]

Hi there,

and sorry for the late answer...

On 06.10.2018 06:26, Bill Deegan wrote:
Strange that it's referring to "/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl",  it should be referring to build/... for the 
unpacked version of the package.


Have you run python bootstrap.py first?

On Fri, Oct 5, 2018 at 1:35 PM Paweł Tomulik mailto:ptomu...@meil.pw.edu.pl>> wrote:

The problem still persists:

I/O warning : failed to load external entity
"/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
scons: *** [build/doc/user/manual.fo ] parserError : 
xmlParseFile() failed
scons: building terminated because of errors.

should I file an issue on GitHub?



I think that xmldepend.xsl should be contained in the packages. I checked back to SCons v2.3.2 and it was never included right from 
the start. Funny that nobody noticed until now, I guess I always tried to rebuild from the sources on my machine...definitely have 
to amend my workflow for preparing release packages. ;)


Pawel, please open an issue on github about this, I'll try to look into it as 
soon as possible.

We should also discuss what to do about the copy of the docbook XSLT stylesheets that are checked in alongside with the sources. 
Should these be packaged too, such that the user doesn't have to care about installing them separately?


Best regards,

Dirk
___
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Re: [Scons-dev] should src/engine/SCons/Tool/docbook/util/* be included in the packages?

[Bill Deegan]

Are the docbook xslt stylesheets we're currently including reasonably
available from distro's repos?


On Sun, Oct 7, 2018 at 2:08 PM Dirk Bächle  wrote:

> Hi there,
>
> and sorry for the late answer...
>
> On 06.10.2018 06:26, Bill Deegan wrote:
> > Strange that it's referring to
> "/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl",  it should be
> referring to build/... for the
> > unpacked version of the package.
> >
> > Have you run python bootstrap.py first?
> >
> > On Fri, Oct 5, 2018 at 1:35 PM Paweł Tomulik  > wrote:
> >
> > The problem still persists:
> >
> > I/O warning : failed to load external entity
> > "/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
> > scons: *** [build/doc/user/manual.fo ]
> parserError : xmlParseFile() failed
> > scons: building terminated because of errors.
> >
> > should I file an issue on GitHub?
> >
>
> I think that xmldepend.xsl should be contained in the packages. I checked
> back to SCons v2.3.2 and it was never included right from
> the start. Funny that nobody noticed until now, I guess I always tried to
> rebuild from the sources on my machine...definitely have
> to amend my workflow for preparing release packages. ;)
>
> Pawel, please open an issue on github about this, I'll try to look into it
> as soon as possible.
>
> We should also discuss what to do about the copy of the docbook XSLT
> stylesheets that are checked in alongside with the sources.
> Should these be packaged too, such that the user doesn't have to care
> about installing them separately?
>
> Best regards,
>
> Dirk
> ___
> Scons-dev mailing list
> Scons-dev@scons.org
> https://pairlist2.pair.net/mailman/listinfo/scons-dev
>
___
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Re: [Scons-dev] should src/engine/SCons/Tool/docbook/util/* be included in the packages?

[Paweł Tomulik]

Gosh, I was working on 3.0.1 (2017, from Debian repo). I've tried with
master and it seems to work. Sorry for the noise.

W dniu 06.10.2018 o 06:26, Bill Deegan pisze:
> Strange that it's referring to
> "/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl",  it should be
> referring to build/... for the unpacked version of the package.
> 
> Have you run python bootstrap.py first?
> 
> On Fri, Oct 5, 2018 at 1:35 PM Paweł Tomulik  > wrote:
> 
> The problem still persists:
> 
> I/O warning : failed to load external entity
> "/usr/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
> scons: *** [build/doc/user/manual.fo ] parserError
> : xmlParseFile() failed
> scons: building terminated because of errors.
> 
> should I file an issue on GitHub?
> 
> W dniu 18.09.2017 o 21:45, Bill Deegan pisze:
> > More data: 
> > Here's the failure output when running against package:
> >
> > warning: failed to load external entity
> >
> 
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl"
> > cannot parse
> >
> 
> /home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Tool/docbook/utils/xmldepend.xsl
> > TypeError: __str__ returned non-string (type NoneType):
> >   File
> >
> 
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 1361:
> >     _exec_main(parser, values)
> >   File
> >
> 
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 1324:
> >     _main(parser)
> >   File
> >
> 
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 1103:
> >     nodes = _build_targets(fs, options, targets, target_top)
> >   File
> >
> 
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 1298:
> >     jobs.run(postfunc = jobs_postfunc)
> >   File
> >
> 
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Job.py",
> > line 111:
> >     self.job.start()
> >   File
> >
> 
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Job.py",
> > line 214:
> >     task.failed()
> >   File
> >
> 
> "/home/bdbaddog/devel/scons/as_scons/build/test-src-tar-gz/lib/scons/SCons/Script/Main.py",
> > line 274:
> >     sys.stderr.write(errfmt % (nodename, buildError))
> >
> >
> > On Mon, Sep 18, 2017 at 3:36 PM, Bill Deegan
> mailto:b...@baddogconsulting.com>
> >  >> wrote:
> >
> >     Dirk,
> >
> >     Looks like it's not currently and when I run "python runtest.py -p
> >     src-tar-gz -a" to run against the generated packages there are
> many
> >     docbook related failures.
> >
> >     Thoughts?
> >
> >     _Bill
> >
> >
> >
> >
> > ___
> > Scons-dev mailing list
> > Scons-dev@scons.org 
> > https://pairlist2.pair.net/mailman/listinfo/scons-dev
> >
> 
> 
> -- 
> Pawel Tomulik
> ___
> Scons-dev mailing list
> Scons-dev@scons.org 
> https://pairlist2.pair.net/mailman/listinfo/scons-dev
> 
> 
> 
> ___
> Scons-dev mailing list
> Scons-dev@scons.org
> https://pairlist2.pair.net/mailman/listinfo/scons-dev
> 


-- 
Pawel Tomulik
___
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Re: [Scons-dev] should src/engine/SCons/Tool/docbook/util/* be included in the packages?

[Dirk Bächle]

Hi Bill,


Am 07.10.2018 um 21:44 schrieb Bill Deegan:
Are the docbook xslt stylesheets we're currently including reasonably 
available from distro's repos?




I haven't checked yet, but my guess is that some version would be 
offered in the major distros. But the problem is then that different 
naming schemes would be used and the folder with the stylesheets would 
be stored in random locations. This makes it difficult to provide an 
out-of-the-box experience with the Docbook Tool.



I saw today that the actual sources of the Docbook XSLT stylesheets are 
now hosted on github too! This made me think: Maybe it would be okay to 
add another "bootstrapping step" to the Docbook Tool, where it clones 
the stylesheets from github to its local folder?


Then we wouldn't have to store all the XSLT files directly in our repo. 
Our build/release workflow would get a little more complicated and users 
of the Docbook Tool would have to "init" it once on each of their 
machines. But maybe it's worth giving it a try?


I could work on a new version of the Docbook Tool that would support 
this, if we agree that's the way to go.



Best regards,


Dirk



___
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Re: [Scons-dev] should src/engine/SCons/Tool/docbook/util/* be included in the packages?

[Bill Deegan]

Assuming we could resolve the issue of locating the XSLT files installed
from native packages, how different would the namespaces be?

break everything?
Or just renamed?

On Mon, Oct 8, 2018 at 2:42 PM Dirk Bächle  wrote:

> Hi Bill,
>
>
> Am 07.10.2018 um 21:44 schrieb Bill Deegan:
> > Are the docbook xslt stylesheets we're currently including reasonably
> > available from distro's repos?
> >
>
> I haven't checked yet, but my guess is that some version would be
> offered in the major distros. But the problem is then that different
> naming schemes would be used and the folder with the stylesheets would
> be stored in random locations. This makes it difficult to provide an
> out-of-the-box experience with the Docbook Tool.
>
>
> I saw today that the actual sources of the Docbook XSLT stylesheets are
> now hosted on github too! This made me think: Maybe it would be okay to
> add another "bootstrapping step" to the Docbook Tool, where it clones
> the stylesheets from github to its local folder?
>
> Then we wouldn't have to store all the XSLT files directly in our repo.
> Our build/release workflow would get a little more complicated and users
> of the Docbook Tool would have to "init" it once on each of their
> machines. But maybe it's worth giving it a try?
>
> I could work on a new version of the Docbook Tool that would support
> this, if we agree that's the way to go.
>
>
> Best regards,
>
>
> Dirk
>
>
>
>
___
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev