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 502278 - jhbuild should reconfigure tarballs when -a specified
jhbuild should reconfigure tarballs when -a specified
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
unspecified
Other All
: Normal enhancement
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
: 365347 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2007-12-07 08:34 UTC by Craig Keogh
Modified: 2007-12-18 15:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
reconfigure tarballs is always autogen is requested. (725 bytes, patch)
2007-12-07 08:36 UTC, Craig Keogh
committed Details | Review
Only unpack tarball if necessary (2.06 KB, patch)
2007-12-07 23:41 UTC, Craig Keogh
none Details | Review
Only unpack tarball if necessary and handle patching (2.49 KB, patch)
2007-12-08 00:24 UTC, Craig Keogh
committed Details | Review

Description Craig Keogh 2007-12-07 08:34:06 UTC
Jhbuild should reconfigure tarballs when -a specified. eg, if the user specified:
jhbuild buildone -a libxklavier
libxklavier should be reconfigured.
Comment 1 Craig Keogh 2007-12-07 08:36:28 UTC
Created attachment 100503 [details] [review]
reconfigure tarballs is always autogen is requested.
Comment 2 Frederic Peters 2007-12-07 09:56:51 UTC
That's fine; do you have commit rights or do you want me to apply it?
Comment 3 Craig Keogh 2007-12-07 10:25:06 UTC
I don't have commit rights. Looking for someone to vouch for me. Volunteering? :)
Comment 4 Craig Keogh 2007-12-07 23:41:28 UTC
Created attachment 100564 [details] [review]
Only unpack tarball if necessary

I suggest apply this patch at the same time as attachment 100503 [details] [review], although the patches are independent.

This patch only unpacks the tarball if the directory doesn't exist. This is useful if the developer has made some changes to the extracted tarball code, and then wants to reconfigure it with the '-a' switch.

The clobber mode still works as before. (Because clobber removes the directory, see method 'do_start' in modtypes/tarball.py)
Comment 5 Craig Keogh 2007-12-08 00:24:52 UTC
Created attachment 100565 [details] [review]
Only unpack tarball if necessary and handle patching

My patch attachment 100564 [details] [review] didn't handle patching of the tarball correctly. With this patch: if the directory already exists, don't unpack and don't re-apply the patches.
Comment 6 Frederic Peters 2007-12-15 20:12:18 UTC
I just commited both patches.

2007-12-15  Frederic Peters  <fpeters@0d.be>

        * jhbuild/modtypes/tarball.py: only unpack tarball if necessary, patch
        by Craig Keogh.

2007-12-15  Frederic Peters  <fpeters@0d.be>

        * jhbuild/modtypes/tarball.py: reconfigure tarballs when -a specified,
        patch by Craig Keogh (closes: #502278)
Comment 7 James Andrewartha 2007-12-18 15:22:59 UTC
*** Bug 365347 has been marked as a duplicate of this bug. ***