pbuild: make sure the files from setup.py's scripts section are copied

[sagetrac-mabshoff]

If one uses pbuild to build the Sage library the files from the scripts section are not copied into $SAGE_ROOT/local/bin:

      scripts = ['sage/dsage/scripts/dsage_worker.py',
                 'sage/dsage/scripts/dsage_setup.py',
                 'spkg-debian-maybe',
                ],

Ergo DSage's doctest just hang at 0% CPU use.

Cheers,

Michael

Component: pbuild

Issue created by migration from https://trac.sagemath.org/ticket/3097

[garyfurnish]

Attachment: trac_3097.patch.gz

[garyfurnish]

comment:3

Attachment: trac_3097_scripts.patch.gz

[sagetrac-mabshoff]

comment:4

The patches will not work as is:

• this patchset, especially the bit to setup.py will break slowbuild
• the scripts repo will be broken once you -sdist
• what happens on clone

This is as-is not going into 3.0.1 :(.

Cheers,

Michael

[garyfurnish]

comment:5

1. The patchset should not break slowbuild (infact the change to slowbuild should fix it for the scripts repo).
2. I don't understand why -sdist is going to break the scripts repo. I added the files to the hg scripts repo, why will sdist break this?
3. the dsage/web directory is a link to devel/sage/...... so it will keep pointing at the right target on clone.

[yqiang]

comment:6

I haven't been able to test this set of patches yet but from speaking with gfurnish on IRC it seems to be the right way to go to resolve the pbuild issue. I'll try these patches out and provide feedback then.

[sagetrac-mabshoff]

Attachment: trac_3097-hgignore.patch.gz

[sagetrac-mabshoff]

Attachment: trac_3097-sdist-fix.patch.gz

Attachment: trac_3097_extcode.patch.gz

this is a slightly updated version of the original patch by Gary

[sagetrac-mabshoff]

Attachment: trac_3097-sdist-fix-part2.patch.gz

oops - this ought to fix the issue

[sagetrac-mabshoff]

comment:8

Ok, now the -sdist issue is resolved. I did an sdist followed by a full rebuild and a testall. Positive review.

Cheers,

Michael

[sagetrac-mabshoff]

comment:9

Merged all six patches in Sage 3.0.2.rc0