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.

Sign up for GitHub

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

Jump to bottom

LinuxDeploy replaced VERSION Environment Variable with LINUXDEPLOY_OUTPUT_VERSION #1361

Closed
rmartin16 opened this issue Jul 14, 2023 · 0 comments · Fixed by #1373
Closed

LinuxDeploy replaced VERSION Environment Variable with LINUXDEPLOY_OUTPUT_VERSION #1361

rmartin16 opened this issue Jul 14, 2023 · 0 comments · Fixed by #1373
Labels
bug A crash or error in behavior. good first issue Is this your first time contributing? This could be a good place to start! linux The issue relates Linux support.

Comments

@rmartin16
Copy link
Member

Describe the bug

LinuxDeploy has replaced VERSION with LINUXDEPLOY_OUTPUT_VERSION; I believe this is because one of the AppImage tools uses VERSION for something else.

For the time being, VERSION will be used as a fallback with the warning below:

[appimage/stderr] Warning: please use $LINUXDEPLOY_OUTPUT_VERSION instead of $VERSION

Steps to reproduce

  1. Run briefcase upgrade linuxdeploy
  2. See output from briefcase build linux appimage

Expected behavior

The new setting is used. Setting both avoids the warning.

Screenshots

No response

Environment

  • Operating System: pop os 22.04
  • Python version: 3.11.4
  • Software versions:
    • Briefcase: 0.3.16.dev11+gede56e67

Logs

No response

Additional context

No response
@rmartin16 rmartin16 added the bug A crash or error in behavior. label Jul 14, 2023
@rmartin16 rmartin16 changed the title LinuxDeploy replace VERSION environment variable with LINUXDEPLOY_OUTPUT_VERSION LinuxDeploy replaced VERSION Environment Variable with LINUXDEPLOY_OUTPUT_VERSION Jul 14, 2023
@rmartin16 rmartin16 added linux The issue relates Linux support. good first issue Is this your first time contributing? This could be a good place to start! labels Jul 14, 2023
@eximious eximious mentioned this issue Jul 22, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug A crash or error in behavior. good first issue Is this your first time contributing? This could be a good place to start! linux The issue relates Linux support.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update Linux deploy to use LINUXDEPLOY_OUTPUT_VERSION eximious/briefcase
1 participant
@rmartin16