Merge lp:~darkmuggle-deactivatedaccount/cloud-init/cloud-init.profile.d into lp:~cloud-init-dev/cloud-init/trunk
Proposed by
Ben Howard
Status: | Merged |
---|---|
Merged at revision: | 553 |
Proposed branch: | lp:~darkmuggle-deactivatedaccount/cloud-init/cloud-init.profile.d |
Merge into: | lp:~cloud-init-dev/cloud-init/trunk |
Diff against target: |
98 lines (+83/-0) 2 files modified
profile.d/Z99-cloud-locale-test.sh (+82/-0) setup.py (+1/-0) |
To merge this branch: | bzr merge lp:~darkmuggle-deactivatedaccount/cloud-init/cloud-init.profile.d |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Scott Moser | Pending | ||
Review via email: mp+100531@code.launchpad.net |
This proposal supersedes a proposal from 2012-03-30.
Description of the change
Added profile.d script to inform users of bad locales.
To post a comment you must log in.
A couple things:
a.) use 'locale 2>&1 | locale_warn' as opposed to the here document.
I was surprised at what was so slow in this, and its the here document. Use a pipe and its much faster.
since we're not modifiying the environment at all, that is ok.
b.) I don't know how i feel about the /var/lib/ cloud/instance/ setting... i think i'd at least like the option of something to be multi-instance, and also would like a per-user setting. as the user who logs in cannot necessarily modify /var/lib/ cloud/instance/ . So just loop over a list of files, that can include the /var/lib/ cloud/instance path
c.) apt-cache search "^language- pack-[a- z][a-z] $"
might be slightly better, as the filtering is then built in.