-
Notifications
You must be signed in to change notification settings - Fork 988
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[bug] meson.configure() passed in the windows style path, which was escaped by libvips #14213
Comments
I have thoroughly debugged the process where this error occurs. The issue arises from the libvips macro conan/conan/tools/meson/meson.py Line 50 in ae001d3
The solution is to either directly replace cmd += ' -Dprefix="{}"'.format(self._conanfile.package_folder.replace('\\', '/')) or in the change to
Should I report this issue to the |
Hi @vcup - Thanks for reporting the issue! I want to be sure if it's either a Conan or a recipe issue. Let's check the most simple case:
[settings]
os=Windows
arch=x86_64
compiler=gcc
compiler.version=10.2
compiler.libcxx=libstdc++11
build_type=Release
[buildenv]
PATH+=(path)C:/msys64/mingw64/bin
CC=C:/msys64/mingw64/bin/gcc.exe
CXX=C:/msys64/mingw64/bin/g++.exe
[runenv]
PATH+=(path)C:/msys64/mingw64/bin
Is it happening that problem again? |
Hello, @franramirez688 - Thanks for your response. Therefore, if I add similar content that causes the issue to a new recipe, the problem will be reproduced. meson.build
main.c #include <config.h>
int main(void) {
return 0;
}
char *issue_func() {
return (MYHELLO_PREFIX);
} Here is the log
And this conan/conan/tools/meson/meson.py Line 50 in ae001d3
Should I re-submit this issue to the libvips repository, or perhaps dealing with the escape character upstream in conan might solve more potential problems? |
Here is minimize reproducible Demo.zip
Run the command |
This was fixed in #14295 for next 2.0.9 release |
* libvips/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericlemanissier.github.io/conan-center-index-bump-deps/) * libvips/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericlemanissier.github.io/conan-center-index-bump-deps/) * libvips/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericlemanissier.github.io/conan-center-index-bump-deps/) * libvips/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericLemanissier.github.io/conan-center-index-bump-deps/) * libvips/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericLemanissier.github.io/conan-center-index-bump-deps/) * libvips/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericLemanissier.github.io/conan-center-index-bump-deps/) * libvips/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericLemanissier.github.io/conan-center-index-bump-deps/) * libvips/all: bump deps Generated and committed by [Conan Center Bump Deps](https://github.com/ericLemanissier/conan-center-index-bump-deps) Find more updatable recipes in the [GitHub Pages](https://ericLemanissier.github.io/conan-center-index-bump-deps/) * workaround conan-io/conan#14213
Environment details
Steps to reproduce
mingw
Logs
Details
The text was updated successfully, but these errors were encountered: