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 414138 - Fails to build when srcdir != builddir
Fails to build when srcdir != builddir
Status: RESOLVED FIXED
Product: gnome-system-tools
Classification: Deprecated
Component: time-admin
CVS latest
Other Linux
: Normal major
: ---
Assigned To: Carlos Garnacho
Carlos Garnacho
Depends on:
Blocks: 414139
 
 
Reported: 2007-03-03 08:32 UTC by Loïc Minier
Modified: 2007-04-09 00:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Search for e-map-marshal.list in $(srcdir)/; add a dependency on e-map-marshal.list to e-map-marshal.c (879 bytes, patch)
2007-03-03 08:33 UTC, Loïc Minier
none Details | Review
Search for e-map-marshal.list in $(srcdir)/; add a dependency on e-map-marshal.list to e-map-marshal.c (take 2) (1.08 KB, patch)
2007-03-03 09:04 UTC, Loïc Minier
none Details | Review
Search for e-map-marshal.list in $(srcdir)/; add a dependency on e-map-marshal.list to e-map-marshal.c (take 3) (1.03 KB, patch)
2007-03-03 09:21 UTC, Loïc Minier
committed Details | Review

Description Loïc Minier 2007-03-03 08:32:05 UTC
Hi,

gnome-system-tools fails to build out of tree with:
Making all in e-map
make[4]: entrant dans le répertoire « /home/lool/jhbuild-gnome-2.18/build/gnome-system-tools/src/time/e-map »
( /home/lool/jhbuild-gnome-2.18/prefix/bin/glib-genmarshal e-map-marshal.list --header > e-map-marshal.tmp \
        && mv e-map-marshal.tmp e-map-marshal.h ) \
        || ( rm -f e-map-marshal.tmp && exit 1 )

(process:1360): GLib-Genmarshal-WARNING **: failed to open "e-map-marshal.list": No such file or directory
make[4]: *** [e-map-marshal.h] Erreur 1
make[4]: quittant le répertoire « /home/lool/jhbuild-gnome-2.18/build/gnome-system-tools/src/time/e-map »

This is due to the reference to the e-map-marshal.list missing a $(srcdir).

I'll attach a patch which fixes and also adds an e-map-marshal.list dependency to the .c (like for the .h).

Bye,
Comment 1 Loïc Minier 2007-03-03 08:33:40 UTC
Created attachment 83787 [details] [review]
Search for e-map-marshal.list in $(srcdir)/; add a dependency on e-map-marshal.list to e-map-marshal.c
Comment 2 Loïc Minier 2007-03-03 09:04:26 UTC
Created attachment 83788 [details] [review]
Search for e-map-marshal.list in $(srcdir)/; add a dependency on e-map-marshal.list to e-map-marshal.c (take 2)

Here's a cleaner patch which will correctly refer to the same .list file in the dependencies as the one it generates; it also uses $< to avoid repeating the path.
Comment 3 Loïc Minier 2007-03-03 09:21:15 UTC
Created attachment 83792 [details] [review]
Search for e-map-marshal.list in $(srcdir)/; add a dependency on e-map-marshal.list to e-map-marshal.c (take 3)

Hmm, using "$<" is actually enough to solve the bug, no need to add $(srcdir) thanks to the VPATH; hence, here's an even simpler patch.

(Sorry for the spam.)
Comment 4 Carlos Garnacho 2007-04-09 00:50:45 UTC
Thanks for the patch, it has been committed to trunk/2.18 :)