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 630726 - Replicate checkoutdir directory trees below buildroot
Replicate checkoutdir directory trees below buildroot
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2010-09-27 12:47 UTC by Dirk Wallenstein
Modified: 2010-11-19 07:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Replicate checkoutdir directory trees below buildroot (1.99 KB, patch)
2010-09-27 12:47 UTC, Dirk Wallenstein
needs-work Details | Review
Replicate checkoutdir directory trees below buildroot (4.09 KB, patch)
2010-09-27 15:37 UTC, Dirk Wallenstein
needs-work Details | Review
Replicate checkoutdir directory trees below buildroot (4.04 KB, patch)
2010-11-18 11:16 UTC, Dirk Wallenstein
committed Details | Review

Description Dirk Wallenstein 2010-09-27 12:47:30 UTC
When using an external buildroot, use the same directory layout as for
checkoutroot. This will prevent clashes when repositories have the same
basename.

Signed-off-by: Dirk Wallenstein <halsmit@t-online.de>
Comment 1 Dirk Wallenstein 2010-09-27 12:47:32 UTC
Created attachment 171199 [details] [review]
Replicate checkoutdir directory trees below buildroot
Comment 2 Dirk Wallenstein 2010-09-27 14:30:19 UTC
Review of attachment 171199 [details] [review]:

Ah! Will create a patch for all modtypes, soon.
Comment 3 Dirk Wallenstein 2010-09-27 15:37:45 UTC
Created attachment 171214 [details] [review]
Replicate checkoutdir directory trees below buildroot

When using an external buildroot, use the same directory layout as for
checkoutroot. This will prevent clashes when repositories have the same
basename. Add this behavior to all modtypes that already support an
external build directory.
Comment 4 Craig Keogh 2010-11-18 10:45:41 UTC
Review of attachment 171214 [details] [review]:

I don't understand the documentation after your change. I think the documentation should be unchanged - after the patch, the directory name is still the directory name where the source lives. Just obtaining the directory name in a more correct manner.
Comment 5 Dirk Wallenstein 2010-11-18 11:10:53 UTC
Yes, that is a bit of a strange text. What I wanted to say is that the full checkoutdir will be used and not just the basename. That makes a difference in Xorg when when the main 'xorg/xorg-server' module gets overridden by another repo 'people/krh/xorg-server'.
Comment 6 Dirk Wallenstein 2010-11-18 11:16:07 UTC
Created attachment 174762 [details] [review]
Replicate checkoutdir directory trees below buildroot

When using an external buildroot, use the same directory layout as for
checkoutroot. This will prevent clashes when repositories have the same
basename. Add this behavior to all modtypes that already support an
external build directory.

Signed-off-by: Dirk Wallenstein <halsmit@t-online.de>
Comment 7 Craig Keogh 2010-11-19 07:37:27 UTC
Comment on attachment 174762 [details] [review]
Replicate checkoutdir directory trees below buildroot

Thank you. Committed.
http://git.gnome.org/browse/jhbuild/commit/?id=c5db5e1dcdc82557528ab76a90da90a84699fc75