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 integrate install stopped working on Linux #27832

Closed
acgetchell opened this issue Nov 15, 2022 · 7 comments · Fixed by microsoft/vcpkg-tool#886
Closed

vcpkg integrate install stopped working on Linux #27832

acgetchell opened this issue Nov 15, 2022 · 7 comments · Fixed by microsoft/vcpkg-tool#886
Assignees
Labels
category:question This issue is a question

Comments

@acgetchell
Copy link

acgetchell commented Nov 15, 2022

Describe the bug
vcpkg integrate install fails

Environment

  • OS: Ubuntu 22.04 LTS
  • Compiler: gcc-11, clang-14

To Reproduce
Steps to reproduce the behavior:

Currently fails on Travis-CI:

https://app.travis-ci.com/github/acgetchell/CDT-plusplus/builds/257819339

Worked 5 days ago:

https://app.travis-ci.com/github/acgetchell/CDT-plusplus/builds/257657548

Same travis.yml file here:

https://github.com/acgetchell/CDT-plusplus/blob/develop/.travis.yml

Expected behavior

I expected it to work as it's done hundreds of times before.

Failure logs

vcpkg integrate install
write_contents("/home/travis/.vcpkg/vcpkg.path.txt"): No such file or directory

https://app.travis-ci.com/github/acgetchell/CDT-plusplus/jobs/588548845#L927

Additional context

I suspect this broke with the update of vcpkg-tool to 2022-11-10.

8430e98

acgetchell added a commit to acgetchell/CDT-plusplus that referenced this issue Nov 15, 2022
vcpkg integrate install not working on Travis-CI, bug report here:

microsoft/vcpkg#27832
@acgetchell
Copy link
Author

Yep, commenting out vcpkg integrate install:

acgetchell/CDT-plusplus@b18b962

Allows the builds to work again:

https://app.travis-ci.com/github/acgetchell/CDT-plusplus/builds/257820170

@LilyWangLL LilyWangLL added the category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) label Nov 16, 2022
@LilyWangLL
Copy link
Contributor

@BillyONeal Could you help take a look this issue? Thanks in advance.

@BillyONeal
Copy link
Member

I'm a bit confused: integrate install is to integrate with Visual Studio and there's little reason to run that on Linux.

@acgetchell
Copy link
Author

Okay. Well, it accidentally worked before, and doesn't now, which seems to be fine.

@BillyONeal
Copy link
Member

Okay. Well, it accidentally worked before, and doesn't now, which seems to be fine.

I mean, just crashing like it looks is happening isn't great either. I was replying not to say 'go away' but to better understand what you're trying to do.

@LilyWangLL LilyWangLL added category:question This issue is a question and removed category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) labels Jan 13, 2023
@LilyWangLL LilyWangLL changed the title vcpkg integrate install stopped working vcpkg integrate install stopped working on Linux Jan 13, 2023
@acgetchell
Copy link
Author

Thank you! I understand you're all very busy so if this is an unsupported use case I'm fine with that. Maybe add something like "looks like you're trying to run this on Linux which is unsupported" to short-circuit this error.

@BillyONeal
Copy link
Member

Thank you! I understand you're all very busy so if this is an unsupported use case I'm fine with that. Maybe add something like "looks like you're trying to run this on Linux which is unsupported" to short-circuit this error.

I just tried this and it didn't repro for me but I added defenses anyway.

@BillyONeal BillyONeal reopened this Feb 7, 2023
BillyONeal added a commit to BillyONeal/vcpkg-tool that referenced this issue Feb 7, 2023
BillyONeal added a commit to microsoft/vcpkg-tool that referenced this issue Feb 9, 2023
* Defend against missing directories in `vcpkg integrate install`. Resolves microsoft/vcpkg#27832

* Consider being unable to get the user configuration home a fatal error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:question This issue is a question
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants