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
l10n.eclass: update l10n_find_plocales_changes() #1308
Conversation
einfo "Looking in '${dir}' for new locales ..." | ||
pushd "${dir}" >/dev/null || die "Cannot access ${dir}" | ||
local current= | ||
for l in ${pre}*${post} ; do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
l
should be local as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much @jlec , didn't know bash doesn't limit scope of loop vars.
This update does - allow to span PLOCALES over multiple lines - make code more readable Gentoo-Bug: 513242
What's holding this back? |
@rindeal: Hi. Eclass changes must be discussed on the gentoo-dev mailing first. Not all Gentoo devs use and/or like Github. Thank you. |
I announced it there on the same day as here, but no one responded and apparently no one read it.
|
I guess this can be closed now. For anyone interested, I've completely reworked the eclass and added many cool features, while keeping full backwards compatibility. It can be seen in action in my overlay - https://github.com/rindeal/gentoo-overlay/blob/master/eclass/l10n.eclass |
This update does
Gentoo-Bug: 513242
@yngwin @jlec