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

g.extension: only warn when non-essential files are missing #164

Merged
merged 1 commit into from
Oct 27, 2019

Conversation

wenzeslaus
Copy link
Member

Only warn, not fatal error, when CSS and logo files are missing
in the distribution. Tell user about the possibility of
missing a doc pkg. Tell what will happen next.

Also improve function and parameter names.
and switch to format and name vars in now more
complex tr string.

This is particularly useful when you don't have grass-doc package installed and you don't want to install it, but want to install modules using g.extension.

Only warn, not fatal error, when CSS and logo files are missing
in the distribution. Tell user about the possibility of
missing a doc pkg. Tell what will happen next.

Also improve function and parameter names.
and switch to format and name vars in now more
complex tr string.
Copy link
Member

@neteler neteler left a comment

Choose a reason for hiding this comment

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

looks good to me

@wenzeslaus wenzeslaus merged commit d076942 into OSGeo:master Oct 27, 2019
@wenzeslaus wenzeslaus deleted the warn-not-fail-addon-install branch October 27, 2019 03:14
neteler pushed a commit that referenced this pull request Oct 27, 2019
Only warn, not fatal error, when CSS and logo files are missing
in the distribution. Tell user about the possibility of
missing a doc pkg. Tell what will happen next.

Also improve function and parameter names.
and switch to format and name vars in now more
complex tr string.
@neteler
Copy link
Member

neteler commented Oct 27, 2019

backported in d8247df

petrasovaa pushed a commit to petrasovaa/grass that referenced this pull request Dec 3, 2019
Only warn, not fatal error, when CSS and logo files are missing
in the distribution. Tell user about the possibility of
missing a doc pkg. Tell what will happen next.

Also improve function and parameter names.
and switch to format and name vars in now more
complex tr string.
landam pushed a commit to landam/grass that referenced this pull request Jan 28, 2020
Only warn, not fatal error, when CSS and logo files are missing
in the distribution. Tell user about the possibility of
missing a doc pkg. Tell what will happen next.

Also improve function and parameter names.
and switch to format and name vars in now more
complex tr string.
@neteler neteler added this to the 7.8.1 milestone Dec 9, 2021
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