diff options
author | Gerald Combs <gerald@wireshark.org> | 2017-01-04 08:28:51 -0800 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2017-01-05 10:01:31 +0000 |
commit | 0782a7a15b4114bbcaa766fcd11ea827bb9107bc (patch) | |
tree | 24b597b031a443e256eb994856201c063f0ae386 | |
parent | 29aa8893ea8fe2efbdb5b9015c840d0982dbaba6 (diff) |
More make-version.pl fixups.
Don't add extra \n's. Change ".*([\r\n]+)$" matches to ".*?([\r\n]+)$"
so that we don't greedily match the wrong line ending.
Change-Id: I916ee49207eaac17e1e4c4f677558f7be13a099a
Reviewed-on: https://code.wireshark.org/review/19541
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
-rwxr-xr-x | make-version.pl | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/make-version.pl b/make-version.pl index 3b693af0f1..36cbf7e750 100755 --- a/make-version.pl +++ b/make-version.pl @@ -420,15 +420,15 @@ sub update_cmakelists_txt open(CFGIN, "< $filepath") || die "Can't read $filepath!"; while ($line = <CFGIN>) { - if ($line =~ /^set *\( *GIT_REVISION .*([\r\n]+)$/) { + if ($line =~ /^set *\( *GIT_REVISION .*?([\r\n]+)$/) { $line = sprintf("set(GIT_REVISION %d)$1", $num_commits); - } elsif ($line =~ /^set *\( *PROJECT_MAJOR_VERSION .*([\r\n]+)$/) { + } elsif ($line =~ /^set *\( *PROJECT_MAJOR_VERSION .*?([\r\n]+)$/) { $line = sprintf("set(PROJECT_MAJOR_VERSION %d)$1", $version_pref{"version_major"}); - } elsif ($line =~ /^set *\( *PROJECT_MINOR_VERSION .*([\r\n]+)$/) { + } elsif ($line =~ /^set *\( *PROJECT_MINOR_VERSION .*?([\r\n]+)$/) { $line = sprintf("set(PROJECT_MINOR_VERSION %d)$1", $version_pref{"version_minor"}); - } elsif ($line =~ /^set *\( *PROJECT_PATCH_VERSION .*([\r\n]+)$/) { + } elsif ($line =~ /^set *\( *PROJECT_PATCH_VERSION .*?([\r\n]+)$/) { $line = sprintf("set(PROJECT_PATCH_VERSION %d)$1", $version_pref{"version_micro"}); - } elsif ($line =~ /^set *\( *PROJECT_VERSION_EXTENSION .*([\r\n]+)$/) { + } elsif ($line =~ /^set *\( *PROJECT_VERSION_EXTENSION .*?([\r\n]+)$/) { $line = sprintf("set(PROJECT_VERSION_EXTENSION \"%s\")$1", $package_string); } $contents .= $line @@ -453,13 +453,13 @@ sub update_configure_ac open(CFGIN, "< $filepath") || die "Can't read $filepath!"; while ($line = <CFGIN>) { - if ($line =~ /^m4_define\( *\[?version_major\]? *,.*([\r\n]+)$/) { + if ($line =~ /^m4_define\( *\[?version_major\]? *,.*?([\r\n]+)$/) { $line = sprintf("m4_define([version_major], [%d])$1", $version_pref{"version_major"}); - } elsif ($line =~ /^m4_define\( *\[?version_minor\]? *,.*([\r\n]+)$/) { + } elsif ($line =~ /^m4_define\( *\[?version_minor\]? *,.*?([\r\n]+)$/) { $line = sprintf("m4_define([version_minor], [%d])$1", $version_pref{"version_minor"}); - } elsif ($line =~ /^m4_define\( *\[?version_micro\]? *,.*([\r\n]+)$/) { + } elsif ($line =~ /^m4_define\( *\[?version_micro\]? *,.*?([\r\n]+)$/) { $line = sprintf("m4_define([version_micro], [%d])$1", $version_pref{"version_micro"}); - } elsif ($line =~ /^m4_define\( *\[?version_extra\]? *,.*([\r\n]+)$/) { + } elsif ($line =~ /^m4_define\( *\[?version_extra\]? *,.*?([\r\n]+)$/) { $line = sprintf("m4_define([version_extra], [%s])$1", $package_string); } $contents .= $line @@ -484,7 +484,7 @@ sub update_attributes_asciidoc while ($line = <ADOC_CONF>) { # :wireshark-version: 2.3.1 - if ($line =~ /^:wireshark-version:.*([\r\n]+)$/) { + if ($line =~ /^:wireshark-version:.*?([\r\n]+)$/) { $line = sprintf(":wireshark-version: %d.%d.%d$1", $version_pref{"version_major"}, $version_pref{"version_minor"}, @@ -511,7 +511,8 @@ sub update_debian_changelog open(CHANGELOG, "< $filepath") || die "Can't read $filepath!"; while ($line = <CHANGELOG>) { if ($set_version && CHANGELOG->input_line_number() == 1) { - $line = sprintf("wireshark (%d.%d.%d) unstable; urgency=low\n", + $line =~ /^.*?([\r\n]+)$/; + $line = sprintf("wireshark (%d.%d.%d) unstable; urgency=low$1", $version_pref{"version_major"}, $version_pref{"version_minor"}, $version_pref{"version_micro"}, @@ -550,7 +551,7 @@ sub update_automake_lib_releases # libwireshark_la_LDFLAGS = -version-info 2:1:1 -export-symbols if ($line =~ /^(lib\w+_la_LDFLAGS.*version-info\s+\d+:)\d+(:\d+.*[\r\n]+)$/) { - $line = sprintf("$1%d$2\n", $version_pref{"version_micro"}); + $line = sprintf("$1%d$2", $version_pref{"version_micro"}); } $contents .= $line } @@ -579,7 +580,7 @@ sub update_cmake_lib_releases # set(FULL_SO_VERSION "0.0.0") if ($line =~ /^(set\s*\(\s*FULL_SO_VERSION\s+"\d+\.\d+\.)\d+(".*[\r\n]+)$/) { - $line = sprintf("$1%d$2\n", $version_pref{"version_micro"}); + $line = sprintf("$1%d$2", $version_pref{"version_micro"}); } $contents .= $line } |