Skip to content

Ian/cxx common action#14

Draft
2over12 wants to merge 35 commits into
masterfrom
ian/cxx-common-action
Draft

Ian/cxx common action#14
2over12 wants to merge 35 commits into
masterfrom
ian/cxx-common-action

Conversation

@2over12

@2over12 2over12 commented Jun 1, 2023

Copy link
Copy Markdown
Contributor

No description provided.

Comment on lines +39 to +48
mono `./vcpkg/vcpkg fetch nuget | tail -n 1` \
sources add \
-source "https://nuget.pkg.github.com/lifting-bits/index.json" \
-storepasswordincleartext \
-name "GitHub" \
-username "lifting-bits" \
-password "${{ inputs.gh-token }}"
mono `./vcpkg/vcpkg fetch nuget | tail -n 1` \
setapikey "${{ inputs.gh-token }}" \
-source "https://nuget.pkg.github.com/lifting-bits/index.json"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very weird it's still not working... In cxx-common, we add first and then update... I forget if that was for a special reason, like self-hosted runners, or if the authentication is finicky
https://github.com/lifting-bits/cxx-common/blob/94533d40904afb26ac979a8c91282f89f3c81fe3/.github/workflows/vcpkg_ci_amd64.yml#L98-L111
also, I run vcpkg from the same shell as the one I authenticate in
https://github.com/lifting-bits/cxx-common/blob/94533d40904afb26ac979a8c91282f89f3c81fe3/.github/workflows/vcpkg_ci_amd64.yml#L117

Comment thread actions/build-cxx-common/action.yml Outdated
working-directory: cxx-common
shell: "bash"
run: >
./vcpkg/vcpkg install sqlite3 cpprestsdk

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
./vcpkg/vcpkg install sqlite3 cpprestsdk
./vcpkg/vcpkg install sqlite3 cpprestsdk --debug

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 this pull request may close these issues.

3 participants