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

Fixing deprecated env vars #3459

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixing deprecated env vars #3459

wants to merge 1 commit into from

Conversation

joedborg
Copy link

No description provided.

@joedborg joedborg self-assigned this Mar 12, 2019
@joedborg joedborg requested a review from evilnick March 12, 2019 14:24
@pmatulis
Copy link

I don't feel we should use JUJU_REPOSITORY. It used to designate a directory for local charms but this has been deprecated since 2.0 afaict. It can still be used as a variable of course but this is confusing. In this context we should use something generic like JUJU_WORK_DIR, if anything at all.

There is also missing one of the more important variables: CHARM_BUILD_DIR.

@joedborg
Copy link
Author

Hey @pmatulis, thanks for the feedback. Would JUJU_WORK_DIR and CHARM_BULD_DIR not need to be the same thing, in this instance?

@evilnick
Copy link
Member

I know we say JUJU_REPOSITORY is deprecated, but it is still referenced in the current docs

https://docs.jujucharms.com/2.5/en/reference-environment-variables#building

which reminds me that any changes here should also be replicated there

@pmatulis
Copy link

@joedborg CHARM_BUILD_DIR has an effect on the system whereas JUJU_WORK_DIR is benign.

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