Please merge sbuild-0.60.9 from Debian unstable

Bug #722159 reported by Tarun K. Mall
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
sbuild (Ubuntu)
Fix Released
Wishlist
Bhavani Shankar

Bug Description

Binary package hint: sbuild

Binary Package: sbuild
New version is available in debian.
Need merging.

Related branches

Tarun K. Mall (c2tarun)
Changed in sbuild (Ubuntu):
assignee: nobody → TARUN KUMAR MALL (c2tarun)
status: New → In Progress
assignee: TARUN KUMAR MALL (c2tarun) → nobody
status: In Progress → Confirmed
Revision history for this message
Tarun K. Mall (c2tarun) wrote :
Revision history for this message
Tarun K. Mall (c2tarun) wrote :
Revision history for this message
Bhavani Shankar (bhavi) wrote :

Hey tarun again

I could not find this change superceeded in the debian version you are merging

   * lib/Sbuild/ResolverBase.pm: use -o APT::Install-Recommends=false
       instead of --no-install-recommends as it is not a valid option in
       Dapper schroots. This is Ubuntu-specific and can be removed once Dapper
       goes EoL.

by

     * When running apt-get, use "-o APT::Install-Recommends=false"
        rather than "--no-install-recommends" in order to support older
        releases (Closes: #609811). Thanks to Marc Deslauriers.

So you should mention in the changelog as a dropped change because its included in debian

Secondly,

I find this change

--- sbuild-0.60.9.orig/bin/sbuild-createchroot
+++ sbuild-0.60.9/bin/sbuild-createchroot
@@ -157,8 +157,7 @@
 # Make sure fakeroot and build-essential are installed
 $conf->set('INCLUDE', add_items($conf->get('INCLUDE'),
     "fakeroot",
- "build-essential",
- "debfoster"));
+ "build-essential"));

 my $suite = $ARGV[0];
 # Create the target directory in advance so abs_path (which is buggy)

which corresponds to this change in the previous merge changelog entries

- bin/sbuild-createchroot: do not install debfoster into the chroots
  because it is in universe and not needed for package building itself.

which is missing in your present merge debdiff above

Please update the changelog correctly

PS: I'm unsubsribing sponsors as I am going to take care of uploading this merge

regards

Changed in sbuild (Ubuntu):
importance: Undecided → Wishlist
status: Confirmed → Incomplete
assignee: nobody → TARUN KUMAR MALL (c2tarun)
Revision history for this message
Bhavani Shankar (bhavi) wrote :

typo above its change is superceeded in debian

regards

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

oops sorry for the comments as I simultaneously looked at the old debdiff you attached here

  https://bugs.launchpad.net/ubuntu/+source/sbuild/+bug/722159/+attachment/1861866/+files/sbuild.debdiff

taking a look now

regards

Changed in sbuild (Ubuntu):
status: Incomplete → In Progress
assignee: TARUN KUMAR MALL (c2tarun) → Bhavani Shankar (bhavi)
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.1 KiB)

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

---------------
sbuild (0.60.9-1ubuntu1) natty; urgency=low

  * Merge from debian unstable (LP: #722159). Remaining change:
    - bin/sbuild-createchroot: do not install debfoster into the chroots
      because it is in universe and not needed for package building itself.
  * Dropped change: (included in debian)
    - lib/Sbuild/ResolverBase.pm: use -o APT::Install-Recommends=false
      instead of --no-install-recommends as it is not a valid option in
      Dapper schroots. This is Ubuntu-specific and can be removed once
      Dapper goes EoL.

sbuild (0.60.9-1) unstable; urgency=low

  * New release.
  * sbuild:
    - Don't automatically generate local archive signing key if
      missing (Closes: #607945). If missing, abort build with an error
      message and instructions for how to generate the key.
    - Do not filter out environment variables starting with DEBIAN_ by
      default (Closes: #608659). Thanks to Modestas Vainius.
    - Do not filter out PATH environment variable (Closes: #610997).
      Note this is not the user's PATH, but the $path in sbuild.conf.
    - Aptitude resolver defines a custom SolutionCost in order to
      improve resolver behaviour (Closes: #608789). Thanks to
      Modestas Vainius.
    - When running apt-get, use "-o APT::Install-Recommends=false"
      rather than "--no-install-recommends" in order to support older
      releases (Closes: #609811). Thanks to Marc Deslauriers.
    - Purging the package removes generated files in /etc/sbuild/chroot
      and /var/lib/sbuild (Closes: #608412).
    - The architecture specified with --arch (defaulting to the host
      system architecture) must match the chroot architecture
      (Closes: #609151). This was previously allowed to be superceded
      by the chroot architecture. This change is to avoid any abiguity
      regarding the host/build architecture for the package being built.
    - Add --no-arch-all option to force not building arch-all packages
      (Closes: #613401). Thanks to Sebastian Harl for this patch.
  * sbuild-createchroot:
    - Use less restrictive permissions on chroot directory and schroot
      configuration file (Closes: #608414). The chroot now has 0755
      permissions, and the schroot configuration 0644.
    - Don't require sudo access (Closes: #608429). The
      Sbuild::ChrootPlain module used to run commands inside the newly-
      created chroot was using sudo to run commands when already
      running as root. sudo is no longer used when root.
    - Remove use statement which prevented command-line options being
      registered (Closes: #610007, #607606). Command-line options now
      work correctly.
  * sbuild.conf:
    - $environment filter is syntactically correct (Closes: #610997).
      Thanks to Jakub Wilk.
    - $apt_allow_unauthenticated is documented to use apt-secure(8)
      (Closes: #608971).
  * sbuild.1:
    - Document the temporary local archive used by the apt and aptitude
      build dependency resolvers to store and install dummy dependency
      packages (Closes: #608972).
 -- Tarun Kumar Mall <email address hidden> Sun, 20 Feb 2011 1...

Read more...

Changed in sbuild (Ubuntu):
status: In Progress → 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.