Please merge sbuild (0.59.0-1)(universe) from debian unstable(main)

Bug #408390 reported by Bhavani Shankar
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
sbuild (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Binary package hint: sbuild

Debian has a new version to be merged

sbuild (0.59.0-1) unstable; urgency=low

  [ Roger Leigh ]
  * New release.
  * Use version comparison functions in WannaBuild::Database
    (Closes: #521486). Thanks to Florian Weimer.
  * WannaBuild::Database: Allow whitespace in Architecture field
    (Closes: #535415). Thanks to John Wright for this patch.
  * Sbuild::Build, WannaBuild::Database: Add support for architecture
    aliases/wildcards (Closes: #501230). Thanks to Andres Mejia for
    these patches.
  * Sbuild::Conf doesn't require STATS_DIR to exist, and
    Sbuild::Build::write_stats only uses STATS_DIR in batch mode
    (Closes: #539570).

 -- Roger Leigh <email address hidden> Sun, 02 Aug 2009 11:15:26 +0100

Related branches

Revision history for this message
Bhavani Shankar (bhavi) wrote :
Changed in sbuild (Ubuntu):
status: New → Confirmed
Revision history for this message
Andrea Veri (av) wrote :

can you please explain me what you did here:

- "build-essential",
- "debfoster"));
+ "build-essential"));

It looks like you forget to modify the maintainer field to MOTU too. (with XSBC stuff too)

Revision history for this message
Andrea Veri (av) wrote :

removing u-u-s as subscriber, gonna take care of this merge request.

Changed in sbuild (Ubuntu):
assignee: nobody → Bhavani Shankar (bhavi)
status: Confirmed → Incomplete
importance: Undecided → Low
importance: Low → Wishlist
Revision history for this message
Andrea Veri (av) wrote :

As I told you before please do NOT copy/paste the previous changelog entry added but write it on your own explaining which change you are applying back and why. Again the bug title is wrong: you merge a package from Debian unstable to Ubuntu universe.

Revision history for this message
Bhavani Shankar (bhavi) wrote :

Hello again andrea,

okay here goes the explanation for your queries

AFAIK sbuild builds packages from your own system under a chrooted environment and as per the code snippet of sbuild-createchroot goes

# Make sure fakeroot and build-essential are installed
$conf->set('INCLUDE', add_items($conf->get('INCLUDE'),
    "fakeroot",
    "build-essential"));
we have to make sure that build essential and fakeroot gets installed in a chrooted environment and debfoster as my understanding goes is used to selectively install packages and here it is not essential to have debfoster as fakeroot and build essential already get included in the build in a chroot environment as per the above code snippet comment requirement so as to make sure that build essential and fakeroot gets installed..

and the package is maintained (in the upstream also) actively and vigorously by debian buildd maintaintainers and there should not be a change I think.

Regards

Revision history for this message
Bhavani Shankar (bhavi) wrote :

And I m following the format

Please merge <package><version>(ubuntu component) from debian<section><debian component> which is alright I think...

Regards

Changed in sbuild (Ubuntu):
status: Incomplete → Confirmed
assignee: Bhavani Shankar (bhavi) → nobody
Revision history for this message
Andrea Veri (av) wrote :

I would prefer to see a title like: merge package-version from debian-unstable to ubuntu-universe, but that's your choice then. I still don't see the debdiff with the fix to the maintainer field. Please fix it.

Changed in sbuild (Ubuntu):
assignee: nobody → Bhavani Shankar (bhavi)
status: Confirmed → Incomplete
Revision history for this message
Bhavani Shankar (bhavi) wrote :

okay andrea here is the updated patch

Regards

Changed in sbuild (Ubuntu):
status: Incomplete → Confirmed
assignee: Bhavani Shankar (bhavi) → nobody
Revision history for this message
Andrea Veri (av) wrote :

Uploaded. Thanks for your work.

Changed in sbuild (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package sbuild - 0.59.0-1ubuntu1

---------------
sbuild (0.59.0-1ubuntu1) karmic; urgency=low

  * Merge from debian unstable, remaining changes: LP: #408390
    - Do not install debfoster into the chroots because it is in universe
      and not needed for package building itself.
  * Modify Maintainer value to match the DebianMaintainerField
    specification.

 -- Bhavani Shankar <email address hidden> Mon, 03 Aug 2009 19:35:15 +0530

Changed in sbuild (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.