Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[bug] conan.tools.gnu.Autotools.install() doesn't work if build machine is Windows #12153

Closed
SpaceIm opened this issue Sep 20, 2022 · 3 comments · Fixed by #12193
Closed

[bug] conan.tools.gnu.Autotools.install() doesn't work if build machine is Windows #12153

SpaceIm opened this issue Sep 20, 2022 · 3 comments · Fixed by #12193
Labels
bug
Milestone
1.54

Comments

@SpaceIm
Copy link
Contributor

SpaceIm commented Sep 20, 2022 •
edited
Loading

Environment Details (include every applicable attribute)

  • Operating System+version: Windows 10
  • Compiler+version: MinGW 9 / gcc 11.2
  • Conan version: 1.52.0
  • Python version: 3.10.2

Steps to reproduce (Include if Applicable)

Fix: convert to unix_path conanfile.package_folder in

conan/conan/tools/gnu/autotools.py

Line 58 in 8aeea22

args = args if args is not None else ["DESTDIR={}".format(self._conanfile.package_folder)]

Logs (Executed commands with output) (Include/Attach if Applicable)

 /usr/bin/install -c -m 644  /c/users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/src/src/liblzma/api/lzma.h 'C:Usersspaceim.conandataxz_utils5.2.5__package3cbc13750c61092b2c900ee6caef68cf1cd320b4/include/.'
make[4]: Leaving directory '/c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/src/liblzma/api'
make[3]: Leaving directory '/c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/src/liblzma/api'
make[3]: Entering directory '/c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/src/liblzma'
make[4]: Entering directory '/c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/src/liblzma'
 /usr/bin/mkdir -p 'C:Usersspaceim.conandataxz_utils5.2.5__package3cbc13750c61092b2c900ee6caef68cf1cd320b4/lib'
 /usr/bin/mkdir -p 'C:Usersspaceim.conandataxz_utils5.2.5__package3cbc13750c61092b2c900ee6caef68cf1cd320b4/lib/pkgconfig'
 /bin/sh ../../libtool   --mode=install /usr/bin/install -c   liblzma.la 'C:Usersspaceim.conandataxz_utils5.2.5__package3cbc13750c61092b2c900ee6caef68cf1cd320b4/lib'
 /usr/bin/install -c -m 644 liblzma.pc 'C:Usersspaceim.conandataxz_utils5.2.5__package3cbc13750c61092b2c900ee6caef68cf1cd320b4/lib/pkgconfig'
Usage: /c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/libtool [OPTION]... [MODE-ARG]...
Try 'libtool --help' for more information.
libtool:   error: 'C:Usersspaceim.conandataxz_utils5.2.5__package3cbc13750c61092b2c900ee6caef68cf1cd320b4/lib' must be an absolute directory name
make[4]: *** [Makefile:876: install-libLTLIBRARIES] Error 1
make[4]: Leaving directory '/c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/src/liblzma'
make[3]: *** [Makefile:1802: install-am] Error 2
make[3]: Leaving directory '/c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/src/liblzma'
make[2]: *** [Makefile:1638: install-recursive] Error 1
make[2]: Leaving directory '/c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/src/liblzma'
make[1]: *** [Makefile:422: install-recursive] Error 1
make[1]: Leaving directory '/c/Users/spaceim/.conan/data/xz_utils/5.2.5/_/_/build/3cbc13750c61092b2c900ee6caef68cf1cd320b4/build-release/src'
make: *** [Makefile:616: install-recursive] Error 1
This was referenced Sep 20, 2022
Closed
Merged
SpaceIm added a commit to SpaceIm/conan-center-index that referenced this issue Sep 20, 2022
@SpaceIm
fix MinGW build
1458445 
DESTDIR manually injected due to conan-io/conan#12153
@SpaceIm SpaceIm mentioned this issue Sep 24, 2022
Merged
5 tasks
conan-center-bot pushed a commit to conan-io/conan-center-index that referenced this issue Sep 28, 2022
@SpaceIm
(#13038) xz_utils: conan v2 support
14eec9c 
* conan v2 support

* typo

* refactor slightly

* fix MinGW build

DESTDIR manually injected due to conan-io/conan#12153

* workaround to update PlatformToolset in vcxproj files

* fix install for compiler=msvc

* remove WindowsTargetPlatformVersion in 5.2.4

* add ugly tricks for MSBuild

* test custom CMake variables in conan generator

variables from https://cmake.org/cmake/help/latest/module/FindLibLZMA.html must be defined

LIBLZMA_HAS_AUTO_DECODER, LIBLZMA_HAS_EASY_ENCODER & LIBLZMA_HAS_LZMA_PRESET are not modeled for the moment

* typo

* set win_bash in build_requirements
kayoub5 pushed a commit to kayoub5/conan-center-index that referenced this issue Sep 29, 2022
@SpaceIm @kayoub5
(conan-io#13038) xz_utils: conan v2 support
c27aec8 
* conan v2 support

* typo

* refactor slightly

* fix MinGW build

DESTDIR manually injected due to conan-io/conan#12153

* workaround to update PlatformToolset in vcxproj files

* fix install for compiler=msvc

* remove WindowsTargetPlatformVersion in 5.2.4

* add ugly tricks for MSBuild

* test custom CMake variables in conan generator

variables from https://cmake.org/cmake/help/latest/module/FindLibLZMA.html must be defined

LIBLZMA_HAS_AUTO_DECODER, LIBLZMA_HAS_EASY_ENCODER & LIBLZMA_HAS_LZMA_PRESET are not modeled for the moment

* typo

* set win_bash in build_requirements
System-Arch pushed a commit to System-Arch/conan-center-index that referenced this issue Oct 7, 2022
@SpaceIm @marc-mw
(conan-io#13038) xz_utils: conan v2 support
b51c5fd 
* conan v2 support

* typo

* refactor slightly

* fix MinGW build

DESTDIR manually injected due to conan-io/conan#12153

* workaround to update PlatformToolset in vcxproj files

* fix install for compiler=msvc

* remove WindowsTargetPlatformVersion in 5.2.4

* add ugly tricks for MSBuild

* test custom CMake variables in conan generator

variables from https://cmake.org/cmake/help/latest/module/FindLibLZMA.html must be defined

LIBLZMA_HAS_AUTO_DECODER, LIBLZMA_HAS_EASY_ENCODER & LIBLZMA_HAS_LZMA_PRESET are not modeled for the moment

* typo

* set win_bash in build_requirements
@madebr
Copy link
Contributor

madebr commented Oct 19, 2022

I stumbled upon this when trying to switch to Autotools for conan-io/conan-center-index#13564.

@czoido czoido added this to the 1.54 milestone Oct 19, 2022
@czoido czoido added the bug label Oct 19, 2022
@memsharded memsharded modified the milestones: 1.54, 1.55 Nov 2, 2022
@czoido czoido modified the milestones: 1.55, 1.54 Nov 2, 2022
@franramirez688
Copy link
Contributor

Hi @SpaceIm

I'm trying to reproduce the issue (and the test from the PR linked), and I was wondering what your mingw_profile looks like. I think it could help us to get it locally.

@franramirez688
Copy link
Contributor

Closed by #12193
It will be released in the next Conan 1.54 version.

SpaceIm added a commit to SpaceIm/conan-center-index that referenced this issue Feb 6, 2023
@SpaceIm
modernize more
8a8697a 
- use new stdcpp_library() instead of legacy one (since conan 1.54.0)
- write custom _sha256sum() function to avoid relying on legacy conans.tools.sha256sum() conan function
- remove workaround for conan-io/conan#12642 (fixed in conan 1.57.0)
- remove workaround for conan-io/conan#12153 (fixed in conan 1.54.0)
@SpaceIm SpaceIm mentioned this issue Feb 6, 2023
Merged
3 tasks
conan-center-bot pushed a commit to conan-io/conan-center-index that referenced this issue Feb 7, 2023
@SpaceIm
(#15778) icu: modernize more for conan v2
a053fb6 
- use new stdcpp_library() instead of legacy one (since conan 1.54.0)
- write custom _sha256sum() function to avoid relying on legacy conans.tools.sha256sum() conan function
- remove workaround for conan-io/conan#12642 (fixed in conan 1.57.0)
- remove workaround for conan-io/conan#12153 (fixed in conan 1.54.0)
sabelka pushed a commit to sabelka/conan-center-index that referenced this issue Feb 12, 2023
@SpaceIm @sabelka
(conan-io#15778) icu: modernize more for conan v2
e616224 
- use new stdcpp_library() instead of legacy one (since conan 1.54.0)
- write custom _sha256sum() function to avoid relying on legacy conans.tools.sha256sum() conan function
- remove workaround for conan-io/conan#12642 (fixed in conan 1.57.0)
- remove workaround for conan-io/conan#12153 (fixed in conan 1.54.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.54
Development

Successfully merging a pull request may close this issue.

Fix autotools install on Windows SpaceIm/conan
5 participants
@madebr @franramirez688 @czoido @memsharded @SpaceIm