contrib/buildsystems: fix expat library name for generated vcxproj

[orgads]

expat.lib -> libexpat.lib (libexpatd.lib for debug build).

Signed-off-by: Orgad Shaneh orgads@gmail.com

cc: Johannes Schindelin Johannes.Schindelin@gmx.de

[orgads]

/submit

[gitgitgadget]

Submitted as pull.728.git.1599458405948.gitgitgadget@gmail.com

To fetch this version into FETCH_HEAD:

git fetch https://github.com/gitgitgadget/git pr-728/orgads/vcxproj-v1

To fetch this version to local tag pr-728/orgads/vcxproj-v1:

git fetch --no-tags https://github.com/gitgitgadget/git tag pr-728/orgads/vcxproj-v1

[gitgitgadget]

On the Git mailing list, Johannes Schindelin wrote (reply to this):

Hi Orgad,

On Mon, 7 Sep 2020, Orgad Shaneh via GitGitGadget wrote:

> From: Orgad Shaneh <orgads@gmail.com>
>
> expat.lib -> libexpat.lib (libexpatd.lib for debug build).

Good point, I forgot to verify that this is also needed for `vcxproj`, not
only for `MSVC=1`...

Thanks,
Dscho

>
> Signed-off-by: Orgad Shaneh <orgads@gmail.com>
> ---
>     contrib/buildsystems: fix expat library name for generated vcxproj
>
>     expat.lib -> libexpat.lib (libexpatd.lib for debug build).
>
>     Signed-off-by: Orgad Shaneh orgads@gmail.com [orgads@gmail.com]
>
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-728%2Forgads%2Fvcxproj-v1
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-728/orgads/vcxproj-v1
> Pull-Request: https://github.com/gitgitgadget/git/pull/728
>
>  contrib/buildsystems/Generators/Vcxproj.pm | 1 +
>  contrib/buildsystems/engine.pl             | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/contrib/buildsystems/Generators/Vcxproj.pm b/contrib/buildsystems/Generators/Vcxproj.pm
> index 5c666f9ac0..d2584450ba 100644
> --- a/contrib/buildsystems/Generators/Vcxproj.pm
> +++ b/contrib/buildsystems/Generators/Vcxproj.pm
> @@ -80,6 +80,7 @@ sub createProject {
>        $libs_release = join(";", sort(grep /^(?!libgit\.lib|xdiff\/lib\.lib|vcs-svn\/lib\.lib)/, @{$$build_structure{"$prefix${name}_LIBS"}}));
>        $libs_debug = $libs_release;
>        $libs_debug =~ s/zlib\.lib/zlibd\.lib/g;
> +      $libs_debug =~ s/libexpat\.lib/libexpatd\.lib/g;
>        $libs_debug =~ s/libcurl\.lib/libcurl-d\.lib/g;
>      }
>
> diff --git a/contrib/buildsystems/engine.pl b/contrib/buildsystems/engine.pl
> index 070978506a..2ff9620459 100755
> --- a/contrib/buildsystems/engine.pl
> +++ b/contrib/buildsystems/engine.pl
> @@ -349,7 +349,7 @@ sub handleLinkLine
>          } elsif ("$part" eq "-lcurl") {
>              push(@libs, "libcurl.lib");
>          } elsif ("$part" eq "-lexpat") {
> -            push(@libs, "expat.lib");
> +            push(@libs, "libexpat.lib");
>          } elsif ("$part" eq "-liconv") {
>              push(@libs, "libiconv.lib");
>          } elsif ($part =~ /^[-\/]/) {
>
> base-commit: e19713638985533ce461db072b49112da5bd2042
> --
> gitgitgadget
>

[gitgitgadget]

User Johannes Schindelin <Johannes.Schindelin@gmx.de> has been added to the cc: list.

[gitgitgadget]

This branch is now known as os/vcbuild.

[gitgitgadget]

This patch series was integrated into seen via git@b3bb9da.

[gitgitgadget]

This patch series was integrated into next via git@5655140.

[gitgitgadget]

This patch series was integrated into seen via git@ff75e6c.

[gitgitgadget]

This patch series was integrated into next via git@ff75e6c.

[gitgitgadget]

This patch series was integrated into master via git@ff75e6c.

[gitgitgadget]

Closed via ff75e6c.