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

refact: Replace deprecated Snapcraft arch env variables #197

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

locnnil
Copy link
Contributor

@locnnil locnnil commented Dec 13, 2024

Fix Warnings related to Snapcraft environment variables during building of snap:

CRAFT_ARCH_TRIPLET is deprecated, use CRAFT_ARCH_TRIPLET_BUILD_{ON|FOR}
CRAFT_ARCH_TRIPLET is deprecated, use CRAFT_ARCH_TRIPLET_BUILD_{ON|FOR}
CRAFT_ARCH_TRIPLET is deprecated, use CRAFT_ARCH_TRIPLET_BUILD_{ON|FOR}
CRAFT_TARGET_ARCH is deprecated, use CRAFT_ARCH_BUILD_FOR
CRAFT_TARGET_ARCH is deprecated, use CRAFT_ARCH_BUILD_FOR
CRAFT_TARGET_ARCH is deprecated, use CRAFT_ARCH_BUILD_FOR
CRAFT_TARGET_ARCH is deprecated, use CRAFT_ARCH_BUILD_FOR
CRAFT_TARGET_ARCH is deprecated, use CRAFT_ARCH_BUILD_FOR
CRAFT_TARGET_ARCH is deprecated, use CRAFT_ARCH_BUILD_FOR
CRAFT_TARGET_ARCH is deprecated, use CRAFT_ARCH_BUILD_FOR
CRAFT_TARGET_ARCH is deprecated, use CRAFT_ARCH_BUILD_FOR
CRAFT_ARCH_TRIPLET is deprecated, use CRAFT_ARCH_TRIPLET_BUILD_{ON|FOR}
CRAFT_ARCH_TRIPLET is deprecated, use CRAFT_ARCH_TRIPLET_BUILD_{ON|FOR}

As recommended on: https://snapcraft.io/docs/parts-environment-variables

@locnnil locnnil mentioned this pull request Dec 13, 2024
@locnnil locnnil requested a review from farshidtz December 13, 2024 19:17
@farshidtz farshidtz changed the title fix: Snapcraft env related warnings refact: Replace deprecated Snapcraft arch env variables Dec 16, 2024
@farshidtz farshidtz merged commit 4025064 into canonical:main Dec 16, 2024
2 checks passed
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.

2 participants