After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 340995 - Several build problems (actually 1.1.16)
Several build problems (actually 1.1.16)
Status: RESOLVED FIXED
Product: libxslt
Classification: Platform
Component: general
1.1.15
Other Linux
: Normal normal
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
: 340585 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-05-08 09:43 UTC by Peter Breitenlohner
Modified: 2006-10-11 19:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fix DESTDIR (568 bytes, patch)
2006-05-08 09:44 UTC, Peter Breitenlohner
none Details | Review
fix header comments (1.37 KB, patch)
2006-05-08 09:45 UTC, Peter Breitenlohner
none Details | Review
make test-output more readable (749 bytes, patch)
2006-05-08 09:45 UTC, Peter Breitenlohner
none Details | Review
python handling (6.00 KB, patch)
2006-05-08 09:46 UTC, Peter Breitenlohner
none Details | Review

Description Peter Breitenlohner 2006-05-08 09:43:56 UTC
(1) A staged install by non-root fails due to a missing DESTDIR

See attached patch

(2) The headers contain misleading comments about version numbers

See attached patch

(3) The test-output is hard to read (missing linebreaks)

See attached patch

(4) The python detection is fragile and contains invalid shell syntax. Why not
let automake do most of the work. Don't record build directory in installed .la
files

See attached patch
Comment 1 Peter Breitenlohner 2006-05-08 09:44:52 UTC
Created attachment 65003 [details] [review]
fix DESTDIR
Comment 2 Peter Breitenlohner 2006-05-08 09:45:21 UTC
Created attachment 65004 [details] [review]
fix header comments
Comment 3 Peter Breitenlohner 2006-05-08 09:45:59 UTC
Created attachment 65005 [details] [review]
make test-output more readable
Comment 4 Peter Breitenlohner 2006-05-08 09:46:37 UTC
Created attachment 65006 [details] [review]
python handling
Comment 5 Daniel Veillard 2006-10-11 19:36:56 UTC
Patch #1: right applied, thanks
Patch #2: also applied
Patch #3: I like having the output in line, sorry no
Patch #4: AM_PATH_PYTHON not available on all platforms, already revisited,
          sorry no, but I applied the libxsltmod_la_LDFLAGS change

Those applied are now commited in CVS,

 thanks !

Daniel
Comment 6 Daniel Veillard 2006-10-11 19:39:06 UTC
*** Bug 340585 has been marked as a duplicate of this bug. ***