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.

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

[vcpkg] vcpkg build returns: "Value was null" #12355

Closed
Neumann-A opened this issue Jul 10, 2020 · 7 comments · Fixed by #12367
Closed

[vcpkg] vcpkg build returns: "Value was null" #12355

Neumann-A opened this issue Jul 10, 2020 · 7 comments · Fixed by #12367

Comments

@Neumann-A
Copy link
Contributor

neumann@Iluinrandir:/mnt/e/qt$ ./vcpkg build fontconfig --debug
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'manifests' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Using vcpkg-root: /mnt/e/qt
[DEBUG] Using installed-root: /mnt/e/qt/installed
[DEBUG] Using buildtrees-root: /mnt/e/qt/buildtrees
[DEBUG] Using downloads-root: /mnt/d/vcpkg_common/downloads
[DEBUG] Using packages-root: /mnt/e/qt/packages
[DEBUG] Using scripts-root: /mnt/e/qt/scripts
[DEBUG] Found path: /usr/bin/cmake
[DEBUG] Found path: /bin/cmake
[DEBUG] popen("/usr/bin/cmake" --version 2>&1)
[DEBUG] cmd_execute_and_stream_data() returned 0 after     2761 us
[DEBUG] popen(/usr/bin/cmake -DVCPKG_ROOT_DIR=/mnt/e/qt -DPACKAGES_DIR=/mnt/e/qt/packages -DBUILDTREES_DIR=/mnt/e/qt/buildtrees -D_VCPKG_INSTALLED_DIR=/mnt/e/qt/installed -DDOWNLOADS=/mnt/d/vcpkg_common/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P /mnt/e/qt/buildtrees/0.vcpkg_dep_info.cmake 2>&1)
[DEBUG] cmd_execute_and_stream_data() returned 0 after     8447 us
[DEBUG] popen(/usr/bin/cmake -DVCPKG_ROOT_DIR=/mnt/e/qt -DPACKAGES_DIR=/mnt/e/qt/packages -DBUILDTREES_DIR=/mnt/e/qt/buildtrees -D_VCPKG_INSTALLED_DIR=/mnt/e/qt/installed -DDOWNLOADS=/mnt/d/vcpkg_common/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P /mnt/e/qt/buildtrees/0.vcpkg_tags.cmake 2>&1)
[DEBUG] cmd_execute_and_stream_data() returned 0 after     9130 us
Value was null
[DEBUG] ../src/vcpkg/build.cpp(920)
[DEBUG] Exiting after 162894 us (162894 us)
@Neumann-A
Copy link
Contributor Author

Something is not right: passing --editable does not work

neumann@Iluinrandir:/mnt/e/qt$ ./vcpkg install fontconfig --editable
Computing installation plan...
The following packages will be built and installed:
  * brotli[core]:x64-linux
    fontconfig[core]:x64-linux
  * freetype[bzip2,core,png]:x64-linux
  * libiconv[core]:x64-linux
  * pthread[core]:x64-linux
  * pthreads[core]:x64-linux
Additional packages (*) will be modified to complete this operation.
Starting package 1/6: brotli:x64-linux
Building package brotli[core]:x64-linux...
-- Using cached /mnt/d/vcpkg_common/downloads/google-brotli-v1.0.7.tar.gz
-- Cleaning sources at /mnt/e/qt/buildtrees/brotli/src/v1.0.7-bf01b84227.clean. Pass --editable to vcpkg to reuse sources.
-- Extracting source /mnt/d/vcpkg_common/downloads/google-brotli-v1.0.7.tar.gz
-- Applying patch install.patch
-- Applying patch fix-arm-uwp.patch
-- Using source at /mnt/e/qt/buildtrees/brotli/src/v1.0.7-bf01b84227.clean
-- Configuring x64-linux-dbg

@Neumann-A
Copy link
Contributor Author

@ras0219 @ras0219-msft @strega-nil : Please fix ;)

@strega-nil
Copy link
Contributor

I thought we fixed this bug, is it broken again??? Are you sure you're on latest master?

@Neumann-A
Copy link
Contributor Author

@strega-nil: I rebuild vcpkg ~7 hours ago after a merge with master so I assume yes. Checking files.cpp and it is missing the work_queue include which was changed in the latest master commit.

@strega-nil
Copy link
Contributor

Alright, lemme check it out.

@ras0219
Copy link
Contributor

ras0219 commented Jul 10, 2020

@Neumann-A In your editable example, only the explicit (cli) packages are marked "editable" -- transitive dependencies will not be marked. I see that the message "Pass --editable to vcpkg to reuse sources." is super misleading here; I'll open a PR to make that clearer :)

@ras0219
Copy link
Contributor

ras0219 commented Jul 10, 2020

Opened #12366

strega-nil added a commit to strega-nil/vcpkg that referenced this issue Jul 10, 2020
strega-nil added a commit that referenced this issue Jul 10, 2020
strega-nil added a commit to strega-nil/vcpkg that referenced this issue May 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants