Doug, note that the WPKG_ARCHIVE variable set by cmake is the one used to determine the OS and processor to be used by default (and thus in ${architecture()}). So Mathieu is correct. The CMakeLists.txt file needs to be fixed in that regard. And also indeed it should use "mswindows". So testing with WPKG_OS set to "mswindows" should work as is. What we use in the code is the MO_WINDOWS and MO_MING64 anyway, so it should have other side effects.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Ah hah! I get what you're saying. All I have to do is realize that wpkg
is built using CMake. :) Even though I pretty much wrote the cmake
scripts, right? :)
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
He! He! Well... I think I am the one who added those variables to the cmake script. You can do a grep, but I don't think we use the WPKG_OS variable for anything more than that WPKG_ARCHIVE variable in cmake, so we can set it to pretty much anything and it won't affect the compile results other than the default archive.
Last edit: Alexis Wilke 2015-03-12
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Hello,
These changes in the CMakeLists.txt file are a bit annoying with source packages:
Because when we use
In the control.info file of a source package, it sets the value to mingw64-amd64 (for example) and WPKG doesn't support this architecture.
Thanks
Regards
We need to change this in wpkg itself. I agree that it should use
"mswindows" instead of "mingw64."
Doug, note that the WPKG_ARCHIVE variable set by cmake is the one used to determine the OS and processor to be used by default (and thus in ${architecture()}). So Mathieu is correct. The CMakeLists.txt file needs to be fixed in that regard. And also indeed it should use "mswindows". So testing with WPKG_OS set to "mswindows" should work as is. What we use in the code is the MO_WINDOWS and MO_MING64 anyway, so it should have other side effects.
Ah hah! I get what you're saying. All I have to do is realize that wpkg
is built using CMake. :) Even though I pretty much wrote the cmake
scripts, right? :)
He! He! Well... I think I am the one who added those variables to the cmake script. You can do a grep, but I don't think we use the WPKG_OS variable for anything more than that WPKG_ARCHIVE variable in cmake, so we can set it to pretty much anything and it won't affect the compile results other than the default archive.
Last edit: Alexis Wilke 2015-03-12
Here the trivial patch...
Regards
Okay, I pushed this one too along the VERBOSE and CD /D patches.
Thank you Alexis