The following commit has been merged in the master branch:
commit 9f2c48ff8d3c113d627e799650e97b6f734e6f93
Author: Raphaël Hertzog <hert...@debian.org>
Date: Mon Aug 1 22:01:20 2011 +0200
Revert "dpkg-buildpackage: support for Build-Features: build-arch"
This reverts commit 14d48ef9abc2ce2d394e9ae4d69d4ba68b551620. Given
that Guillem does not want this field and that the technical committee
has not yet decided on the proper approach to enable
build-arch/build-indep, it's safer to revert this change for now and allow
us to proceed with a release.
Conflicts:
man/deb-src-control.5
diff --git a/debian/changelog b/debian/changelog
index cc7716a..f195a80 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -177,12 +177,6 @@ dpkg (1.16.1) UNRELEASED; urgency=low
in the m4 DPKG_ARCHITECTURE macro.
* Fix possible segfault of dpkg in findbreakcycle(). LP: #733414
- [ Bill Allombert]
- * Add support for Build-Features: build-arch. Closes: #229357
- With this flag, dpkg-buildpackage will now run "debian/rules build-arch"
- or "debian/rules build-indep" instead of "debian/rules build" when
- appropriate.
-
[ Updated dpkg translations ]
* German (Sven Joachim). Closes: #620312
* Swedish (Peter Krefting).
diff --git a/man/deb-src-control.5 b/man/deb-src-control.5
index de9bec7..c43947e 100644
--- a/man/deb-src-control.5
+++ b/man/deb-src-control.5
@@ -108,13 +108,6 @@ A list of these values can be obtained from the latest
version of the
package.
.TP
-.BI Build\-Features: " features-list"
-A comma-separated list of names of build-time features that are supported
-by the source package. Currently, the only supported feature is
-\fBbuild-arch\fP. It allows dpkg-buildpackage to call the \fIbuild-arch\fP
-or \fIbuild-indep\fP targets in place of \fIbuild\fP.
-
-.TP
.BI Build\-Depends: " package-list"
A list of packages that need to be installed and configured to be able to build
the source package.
diff --git a/man/dpkg-buildpackage.1 b/man/dpkg-buildpackage.1
index cdfcec0..734c696 100644
--- a/man/dpkg-buildpackage.1
+++ b/man/dpkg-buildpackage.1
@@ -27,15 +27,12 @@ It calls \fBdpkg-source \-b\fP to generate the source
package (unless
a binary\-only build has been requested with \fB\-b\fP, \fB\-B\fP or
\fB\-A\fP).
.IP \fB5.\fP 3
-It calls \fBdebian/rules\fP \fIbuild-target\fP followed by
+It calls \fBdebian/rules\fP \fBbuild\fP followed by
\fBfakeroot debian/rules\fP \fIbinary-target\fP (unless a source-only
-build has been requested with \fB\-S\fP). Note that \fIbuild-target\fP
-is \fBbuild\fP unless the control file defines the build-feature
\fBbuild-arch\fP,
-in which case it is either \fBbuild\fP (default case) or \fBbuild-arch\fP (if
-\fB\-B\fP is specified) or \fBbuild-indep\fP (if \fB\-A\fP is specified).
-\fIbinary-target\fR is either \fBbinary\fP (default case) or
-\fBbinary-arch\fP (if \fB\-B\fP is specified) or \fBbinary-indep\fP (if
-\fB\-A\fP is specified).
+build has been requested with \fB\-S\fP). Note that \fIbinary-target\fR is
+either \fBbinary\fP (default case, or if \fB\-b\fP is specified)
+or \fBbinary-arch\fP (if \fB\-B\fP is specified) or \fBbinary-indep\fP
+(if \fB\-A\fP is specified).
.IP \fB6.\fP 3
It calls \fBgpg\fP to sign the \fB.dsc\fP file (if any, unless
\fB\-us\fP is specified).
diff --git a/scripts/Dpkg/BuildFeatures.pm b/scripts/Dpkg/BuildFeatures.pm
deleted file mode 100644
index 6cb254d..0000000
--- a/scripts/Dpkg/BuildFeatures.pm
+++ /dev/null
@@ -1,84 +0,0 @@
-# Copyright © 2007 Frank Lichtenheld <dj...@debian.org>
-# Copyright © 2010-2011 Raphaël Hertzog <hert...@debian.org>
-# Copyright © 2010-2011 Bill Allombert <ballo...@debian.org>
-#
-# This program is free software; you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; either version 2 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-
-package Dpkg::BuildFeatures;
-
-use strict;
-use warnings;
-
-our $VERSION = "0.01";
-
-use Dpkg::Control::Info;
-
-=encoding utf8
-
-=head1 NAME
-
-Dpkg::BuildFeatures - parse the Build-Features control field
-
-=head1 DESCRIPTION
-
-The Dpkg::BuildFeatures object can be used to query the options
-stored in the Build-Features control field.
-
-=head1 FUNCTIONS
-
-=over 4
-
-=item my $bf = Dpkg::BuildFeatures->new($controlfile)
-
-Create a new Dpkg::BuildFeatures object. It will be initialized based
-on the value of the Build-Features control field. If $controlfile is omitted,
it
-loads debian/control.
-
-=cut
-
-sub new {
- my ($this, $controlfile) = @_;
- my $class = ref($this) || $this;
- my $control = Dpkg::Control::Info->new($controlfile);
- my $src_fields = $control->get_source();
- my %buildfeats;
- if (defined($src_fields->{'Build-Features'})) {
- my @buildfeats = split(/\s*,\s*/m, $src_fields->{'Build-Features'});
- %buildfeats = map { $_ => 1 } @buildfeats;
- }
- my $self = { options => \%buildfeats };
- bless $self, $class;
- return $self;
-}
-
-=item $bf->has($option)
-
-Returns a boolean indicating whether the option is stored in the object.
-
-=cut
-
-sub has {
- my ($self, $key) = @_;
- return exists $self->{'options'}{$key};
-}
-
-=back
-
-=head1 AUTHOR
-
-Bill Allombert <ballo...@debian.org>
-
-=cut
-
-1;
diff --git a/scripts/Dpkg/Control/Fields.pm b/scripts/Dpkg/Control/Fields.pm
index 0183281..d7d7dcf 100644
--- a/scripts/Dpkg/Control/Fields.pm
+++ b/scripts/Dpkg/Control/Fields.pm
@@ -83,9 +83,6 @@ our %FIELDS = (
dependency => 'union',
dep_order => 10,
},
- 'Build-Features' => {
- allowed => ALL_SRC,
- },
'Changed-By' => {
allowed => CTRL_FILE_CHANGES,
},
diff --git a/scripts/Makefile.am b/scripts/Makefile.am
index f70350a..91929a2 100644
--- a/scripts/Makefile.am
+++ b/scripts/Makefile.am
@@ -56,7 +56,6 @@ nobase_dist_perllib_DATA = \
Dpkg/Arch.pm \
Dpkg/BuildFlags.pm \
Dpkg/BuildOptions.pm \
- Dpkg/BuildFeatures.pm \
Dpkg/Changelog.pm \
Dpkg/Changelog/Debian.pm \
Dpkg/Changelog/Entry.pm \
diff --git a/scripts/dpkg-buildpackage.pl b/scripts/dpkg-buildpackage.pl
index ee3f0f2..40efa0c 100755
--- a/scripts/dpkg-buildpackage.pl
+++ b/scripts/dpkg-buildpackage.pl
@@ -27,7 +27,6 @@ use Dpkg::Gettext;
use Dpkg::ErrorHandling;
use Dpkg::BuildFlags;
use Dpkg::BuildOptions;
-use Dpkg::BuildFeatures;
use Dpkg::Compression;
use Dpkg::Version;
use Dpkg::Changelog::Parse;
@@ -120,7 +119,6 @@ my $checkbuilddep = 1;
my $signsource = 1;
my $signchanges = 1;
my $binarytarget = 'binary';
-my $buildtarget = 'build';
my $targetarch = my $targetgnusystem = '';
my $call_target = '';
my $call_target_as_root = 0;
@@ -204,21 +202,18 @@ while (@ARGV) {
$include = BUILD_BINARY;
push @changes_opts, '-b';
@checkbuilddep_opts = ();
- $buildtarget = 'build';
$binarytarget = 'binary';
} elsif (/^-B$/) {
build_sourceonly && usageerr(_g("cannot combine %s and %s"), $_, "-S");
$include = BUILD_ARCH_DEP;
push @changes_opts, '-B';
@checkbuilddep_opts = ('-B');
- $buildtarget = 'build-arch';
$binarytarget = 'binary-arch';
} elsif (/^-A$/) {
build_sourceonly && usageerr(_g("cannot combine %s and %s"), $_, "-S");
$include = BUILD_ARCH_INDEP;
push @changes_opts, '-A';
@checkbuilddep_opts = ();
- $buildtarget = 'build-indep';
$binarytarget = 'binary-indep';
} elsif (/^-S$/) {
build_binaryonly && usageerr(_g("cannot combine %s and %s"), build_opt,
"-S");
@@ -252,9 +247,6 @@ if ($noclean) {
$include = BUILD_BINARY if ($include & BUILD_DEFAULT);
}
-my $buildfeats = Dpkg::BuildFeatures->new();
-$buildtarget = 'build' unless $buildfeats->has('build-arch');
-
if ($< == 0) {
warning(_g("using a gain-root-command while being root")) if
(@rootcommand);
} else {
@@ -410,7 +402,7 @@ unless (build_binaryonly) {
chdir($dir) or syserr("chdir $dir");
}
unless (build_sourceonly) {
- withecho(@debian_rules, $buildtarget);
+ withecho(@debian_rules, 'build');
withecho(@rootcommand, @debian_rules, $binarytarget);
}
if ($usepause &&
diff --git a/scripts/po/POTFILES.in b/scripts/po/POTFILES.in
index a79375e..88da318 100644
--- a/scripts/po/POTFILES.in
+++ b/scripts/po/POTFILES.in
@@ -17,7 +17,6 @@ scripts/dpkg-shlibdeps.pl
scripts/dpkg-source.pl
scripts/changelog/debian.pl
scripts/Dpkg/Arch.pm
-scripts/Dpkg/BuildFeatures.pm
scripts/Dpkg/BuildFlags.pm
scripts/Dpkg/BuildOptions.pm
scripts/Dpkg/Compression.pm
--
dpkg's main repository
--
To UNSUBSCRIBE, email to debian-dpkg-cvs-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
