Skip to content

Commit 45c6e23

Browse files
committed
Remove --classic build entirely
The Unix build was the last to retain the classic build scheme. The new unified scheme has matured enough, even though some details may need polishing. Reviewed-by: Rich Salz <[email protected]>
1 parent f863ad0 commit 45c6e23

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

67 files changed

+8
-4921
lines changed

.travis.yml

Lines changed: 6 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@ env:
2626
- CONFIG_OPTS=""
2727
- CONFIG_OPTS="--debug no-shared enable-crypto-mdebug enable-rc5 enable-md2"
2828
- CONFIG_OPTS="--strict-warnings no-shared" BUILDONLY="yes"
29-
- CONFIG_OPTS="--classic no-shared" BUILDONLY="yes"
30-
- CONFIG_OPTS="--classic" BUILDONLY="yes"
3129
- CONFIG_OPTS="no-pic --strict-warnings" BUILDONLY="yes"
3230
- CONFIG_OPTS="no-engine no-shared --strict-warnings" BUILDONLY="yes"
3331

@@ -60,34 +58,24 @@ matrix:
6058
before_script:
6159
- sh .travis-create-release.sh $TRAVIS_OS_NAME
6260
- tar -xvzf _srcdist.tar.gz
63-
- if echo "$CONFIG_OPTS" | grep -e "--classic" >/dev/null; then
64-
srcdir=.;
65-
cd _srcdist;
66-
else
67-
srcdir=../_srcdist;
68-
mkdir _build;
69-
cd _build;
70-
fi
61+
- mkdir _build;
62+
- cd _build;
7163
- if [ "$CC" == i686-w64-mingw32-gcc ]; then
7264
export CROSS_COMPILE=${CC%%gcc}; unset CC;
73-
$srcdir/Configure mingw $CONFIG_OPTS -Wno-pedantic-ms-format;
65+
../_srcdir/Configure mingw $CONFIG_OPTS -Wno-pedantic-ms-format;
7466
elif [ "$CC" == x86_64-w64-mingw32-gcc ]; then
7567
export CROSS_COMPILE=${CC%%gcc}; unset CC;
76-
$srcdir/Configure mingw64 $CONFIG_OPTS -Wno-pedantic-ms-format;
68+
../_srcdir/Configure mingw64 $CONFIG_OPTS -Wno-pedantic-ms-format;
7769
else
7870
if which ccache >/dev/null && [ "$CC" != clang-3.6 ]; then
7971
CC="ccache $CC";
8072
fi;
81-
$srcdir/config $CONFIG_OPTS;
73+
../_srcdir/config $CONFIG_OPTS;
8274
fi
8375
- cd ..
8476

8577
script:
86-
- if echo "$CONFIG_OPTS" | grep -e "--classic" >/dev/null; then
87-
cd _srcdist;
88-
else
89-
cd _build;
90-
fi
78+
- cd _build;
9179
- make
9280
- if [ -z "$BUILDONLY" ]; then
9381
if [ -n "$CROSS_COMPILE" ]; then

Configurations/README

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,8 +99,7 @@ In each table entry, the following keys are significant:
9999
some options. In this case, the first
100100
string in the list is the name of the build
101101
scheme.
102-
Currently recognised build schemes are
103-
"unixmake" and "unified".
102+
Currently recognised build scheme is "unified".
104103
For the "unified" build scheme, this item
105104
*must* be an array with the first being the
106105
word "unified" and the second being a word

Configure

Lines changed: 1 addition & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -458,8 +458,6 @@ my $target="";
458458
$config{options}="";
459459
$config{build_type} = "release";
460460

461-
my $classic = 0;
462-
463461
my @argvcopy=@ARGV;
464462

465463
if (grep /^reconf(igure)?$/, @argvcopy) {
@@ -644,11 +642,7 @@ foreach (@argvcopy)
644642
}
645643
elsif (/^[-+]/)
646644
{
647-
if (/^--classic$/)
648-
{
649-
$classic=1;
650-
}
651-
elsif (/^--prefix=(.*)$/)
645+
if (/^--prefix=(.*)$/)
652646
{
653647
$config{prefix}=$1;
654648
die "Directory given with --prefix MUST be absolute\n"
@@ -904,22 +898,6 @@ $config{shared_ldflag} = "";
904898
$target{build_scheme} = [ $target{build_scheme} ]
905899
if ref($target{build_scheme}) ne "ARRAY";
906900

907-
###### TO BE REMOVED WHEN CLASSIC BUILD IS REMOVED
908-
######
909-
###### If the user has chosen --classic, we give it to them.
910-
###### If they try that with an out-of-source config, we complain.
911-
if ($target{build_scheme}->[0] eq "unified" && $classic) {
912-
die "Can't perform a classic build out of source tree\n"
913-
if $srcdir ne $blddir;
914-
915-
$target{build_scheme} = { unix => [ "unixmake" ],
916-
windows => undef,
917-
VMS => undef } -> {$target{build_scheme}->[1]};
918-
919-
die "Classic mode unavailable on this platform\n"
920-
unless defined($target{build_scheme});
921-
}
922-
923901
my ($builder, $builder_platform, @builder_opts) =
924902
@{$target{build_scheme}};
925903

0 commit comments

Comments
 (0)