* [PATCH 0/1] Improve isar-cfg-localepurge @ 2018-05-29 15:15 claudius.heine.ext 2018-05-29 15:15 ` [PATCH 1/1] meta/isar-cfg-localepurge: move locale configuration inside bb claudius.heine.ext 2018-05-30 11:11 ` [PATCH 0/1] Improve isar-cfg-localepurge Claudius Heine 0 siblings, 2 replies; 6+ messages in thread From: claudius.heine.ext @ 2018-05-29 15:15 UTC (permalink / raw) To: isar-users; +Cc: Claudius Heine From: Claudius Heine <ch@denx.de> Hi, this patchset requires my previous patchset: One-shot configuration packages It improves the UX of the isar-cfg-localepurge package and the customization of the locale. It currently does not include a documentation, but that will be added in a latter version of this patchset. Cheers, Claudius Claudius Heine (1): meta/isar-cfg-localepurge: move locale configuration inside bb .../isar-cfg-localepurge/files/locale.debconf | 12 ------ .../isar-cfg-localepurge/files/locale.gen | 1 - .../isar-cfg-localepurge.bb | 43 +++++++++++++++++-- 3 files changed, 40 insertions(+), 16 deletions(-) delete mode 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.debconf delete mode 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.gen -- 2.17.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/1] meta/isar-cfg-localepurge: move locale configuration inside bb 2018-05-29 15:15 [PATCH 0/1] Improve isar-cfg-localepurge claudius.heine.ext @ 2018-05-29 15:15 ` claudius.heine.ext 2018-05-30 8:23 ` Claudius Heine 2018-05-30 11:11 ` [PATCH 0/1] Improve isar-cfg-localepurge Claudius Heine 1 sibling, 1 reply; 6+ messages in thread From: claudius.heine.ext @ 2018-05-29 15:15 UTC (permalink / raw) To: isar-users; +Cc: Claudius Heine From: Claudius Heine <ch@denx.de> To set custom locale configuration it in necessary to overwrite the files of isar-cfg-localepurge before. That can be a bit of a hassle. With this patch the locale configuration is generated based on bitbake variables and therefor can be set from outside. The new variables are `LOCALE_GEN` which is basically the content of the desired `/etc/locale.gen` with each line seperated by '\n' and `LOCALE_DEFAULT` which sets the global default locale of the system. Signed-off-by: Claudius Heine <ch@denx.de> --- .../isar-cfg-localepurge/files/locale.debconf | 12 ------ .../isar-cfg-localepurge/files/locale.gen | 1 - .../isar-cfg-localepurge.bb | 43 +++++++++++++++++-- 3 files changed, 40 insertions(+), 16 deletions(-) delete mode 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.debconf delete mode 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.gen diff --git a/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf b/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf deleted file mode 100644 index 169c071..0000000 --- a/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf +++ /dev/null @@ -1,12 +0,0 @@ -locales locales/locales_to_be_generated multiselect en_US.UTF-8 UTF-8 -locales locales/default_environment_locale select en_US.UTF-8 - -localepurge localepurge/dontbothernew boolean true -localepurge localepurge/nopurge multiselect en, en_US, en_US.UTF-8 -localepurge localepurge/use-dpkg-feature boolean false -localepurge localepurge/verbose boolean false -localepurge localepurge/showfreedspace boolean false -localepurge localepurge/none_selected boolean false -localepurge localepurge/mandelete boolean true -localepurge localepurge/quickndirtycalc boolean false -localepurge localepurge/remove_no note diff --git a/meta/recipes-support/isar-cfg-localepurge/files/locale.gen b/meta/recipes-support/isar-cfg-localepurge/files/locale.gen deleted file mode 100644 index a66d814..0000000 --- a/meta/recipes-support/isar-cfg-localepurge/files/locale.gen +++ /dev/null @@ -1 +0,0 @@ -en_US.UTF-8 UTF-8 diff --git a/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb b/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb index d2cf1f3..f815717 100644 --- a/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb +++ b/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb @@ -4,12 +4,49 @@ DESCRIPTION = "Isar configuration package for locale and localepurge" MAINTAINER = "isar-users <isar-users@googlegroups.com>" DEBIAN_DEPENDS = "localepurge" -SRC_URI = "file://locale.debconf \ - file://locale.gen \ - file://postinst" +SRC_URI = "file://postinst" inherit dpkg-raw +LOCALE_GEN ?= "en_US.UTF-8 UTF-8\n\ + en_US ISO-8859-1\n" +LOCALE_DEFAULT ?= "en_US.UTF-8" + +def get_locale_gen(d): + locale_gen = d.getVar("LOCALE_GEN", True) or "" + return ', '.join(sorted(set(i.strip() + for i in locale_gen.split('\\n')))) + +def get_nopurge(d): + locale_gen = d.getVar("LOCALE_GEN", True) or "" + return ', '.join(sorted(set(i.strip() + for j in locale_gen.split('\\n') + for i in (j.split()[0].split("_")[0], + j.split()[0].split(".")[0], + j.split()[0])))) + +do_gen_config[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}" +do_gen_config() { + cat<<-__EOF__ > ${WORKDIR}/locale.gen + ${@(d.getVar("LOCALE_GEN", True) or "").replace('\\n','\n')} + __EOF__ + cat<<-__EOF__ > ${WORKDIR}/locale.debconf + locales locales/locales_to_be_generated multiselect ${@get_locale_gen(d)} + locales locales/default_environment_locale select ${LOCALE_DEFAULT} + + localepurge localepurge/dontbothernew boolean true + localepurge localepurge/nopurge multiselect ${@get_nopurge(d)} + localepurge localepurge/use-dpkg-feature boolean false + localepurge localepurge/verbose boolean false + localepurge localepurge/showfreedspace boolean false + localepurge localepurge/none_selected boolean false + localepurge localepurge/mandelete boolean true + localepurge localepurge/quickndirtycalc boolean false + localepurge localepurge/remove_no note + __EOF__ +} +addtask gen_config after do_unpack before do_install + do_install() { install -v -d ${D}/usr/lib/${PN} install -v -m 644 ${WORKDIR}/locale.debconf \ -- 2.17.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] meta/isar-cfg-localepurge: move locale configuration inside bb 2018-05-29 15:15 ` [PATCH 1/1] meta/isar-cfg-localepurge: move locale configuration inside bb claudius.heine.ext @ 2018-05-30 8:23 ` Claudius Heine 2018-05-30 8:38 ` Maxim Yu. Osipov 0 siblings, 1 reply; 6+ messages in thread From: Claudius Heine @ 2018-05-30 8:23 UTC (permalink / raw) To: isar-users; +Cc: Claudius Heine Hi, sorry, but I messed this patch up. I will fix it an repost it again with documentation. Claudius On 2018-05-29 17:15, [ext] claudius.heine.ext@siemens.com wrote: > From: Claudius Heine <ch@denx.de> > > To set custom locale configuration it in necessary to overwrite the > files of isar-cfg-localepurge before. That can be a bit of a hassle. > > With this patch the locale configuration is generated based on bitbake > variables and therefor can be set from outside. > > The new variables are `LOCALE_GEN` which is basically the content of the > desired `/etc/locale.gen` with each line seperated by '\n' and > `LOCALE_DEFAULT` which sets the global default locale of the system. > > Signed-off-by: Claudius Heine <ch@denx.de> > --- > .../isar-cfg-localepurge/files/locale.debconf | 12 ------ > .../isar-cfg-localepurge/files/locale.gen | 1 - > .../isar-cfg-localepurge.bb | 43 +++++++++++++++++-- > 3 files changed, 40 insertions(+), 16 deletions(-) > delete mode 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.debconf > delete mode 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.gen > > diff --git a/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf b/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf > deleted file mode 100644 > index 169c071..0000000 > --- a/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf > +++ /dev/null > @@ -1,12 +0,0 @@ > -locales locales/locales_to_be_generated multiselect en_US.UTF-8 UTF-8 > -locales locales/default_environment_locale select en_US.UTF-8 > - > -localepurge localepurge/dontbothernew boolean true > -localepurge localepurge/nopurge multiselect en, en_US, en_US.UTF-8 > -localepurge localepurge/use-dpkg-feature boolean false > -localepurge localepurge/verbose boolean false > -localepurge localepurge/showfreedspace boolean false > -localepurge localepurge/none_selected boolean false > -localepurge localepurge/mandelete boolean true > -localepurge localepurge/quickndirtycalc boolean false > -localepurge localepurge/remove_no note > diff --git a/meta/recipes-support/isar-cfg-localepurge/files/locale.gen b/meta/recipes-support/isar-cfg-localepurge/files/locale.gen > deleted file mode 100644 > index a66d814..0000000 > --- a/meta/recipes-support/isar-cfg-localepurge/files/locale.gen > +++ /dev/null > @@ -1 +0,0 @@ > -en_US.UTF-8 UTF-8 > diff --git a/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb b/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb > index d2cf1f3..f815717 100644 > --- a/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb > +++ b/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb > @@ -4,12 +4,49 @@ DESCRIPTION = "Isar configuration package for locale and localepurge" > MAINTAINER = "isar-users <isar-users@googlegroups.com>" > DEBIAN_DEPENDS = "localepurge" > > -SRC_URI = "file://locale.debconf \ > - file://locale.gen \ > - file://postinst" > +SRC_URI = "file://postinst" > > inherit dpkg-raw > > +LOCALE_GEN ?= "en_US.UTF-8 UTF-8\n\ > + en_US ISO-8859-1\n" > +LOCALE_DEFAULT ?= "en_US.UTF-8" > + > +def get_locale_gen(d): > + locale_gen = d.getVar("LOCALE_GEN", True) or "" > + return ', '.join(sorted(set(i.strip() > + for i in locale_gen.split('\\n')))) > + > +def get_nopurge(d): > + locale_gen = d.getVar("LOCALE_GEN", True) or "" > + return ', '.join(sorted(set(i.strip() > + for j in locale_gen.split('\\n') > + for i in (j.split()[0].split("_")[0], > + j.split()[0].split(".")[0], > + j.split()[0])))) > + > +do_gen_config[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}" > +do_gen_config() { > + cat<<-__EOF__ > ${WORKDIR}/locale.gen > + ${@(d.getVar("LOCALE_GEN", True) or "").replace('\\n','\n')} > + __EOF__ > + cat<<-__EOF__ > ${WORKDIR}/locale.debconf > + locales locales/locales_to_be_generated multiselect ${@get_locale_gen(d)} > + locales locales/default_environment_locale select ${LOCALE_DEFAULT} > + > + localepurge localepurge/dontbothernew boolean true > + localepurge localepurge/nopurge multiselect ${@get_nopurge(d)} > + localepurge localepurge/use-dpkg-feature boolean false > + localepurge localepurge/verbose boolean false > + localepurge localepurge/showfreedspace boolean false > + localepurge localepurge/none_selected boolean false > + localepurge localepurge/mandelete boolean true > + localepurge localepurge/quickndirtycalc boolean false > + localepurge localepurge/remove_no note > + __EOF__ > +} > +addtask gen_config after do_unpack before do_install > + > do_install() { > install -v -d ${D}/usr/lib/${PN} > install -v -m 644 ${WORKDIR}/locale.debconf \ > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] meta/isar-cfg-localepurge: move locale configuration inside bb 2018-05-30 8:23 ` Claudius Heine @ 2018-05-30 8:38 ` Maxim Yu. Osipov 2018-05-30 10:32 ` Claudius Heine 0 siblings, 1 reply; 6+ messages in thread From: Maxim Yu. Osipov @ 2018-05-30 8:38 UTC (permalink / raw) To: Claudius Heine, isar-users; +Cc: Claudius Heine Hi Claudius, Yesterday you've sent this patch over the same codebase you've sent 3 hours before (One shot configuration patch series V5). Is it possible to avoid in the future such frequency of incremental patches as process of testing/applying patches is rather time consuming. Kind regards, Maxim. On 05/30/2018 10:23 AM, Claudius Heine wrote: > Hi, > > sorry, but I messed this patch up. I will fix it an repost it again with > documentation. > > Claudius > > On 2018-05-29 17:15, [ext] claudius.heine.ext@siemens.com wrote: >> From: Claudius Heine <ch@denx.de> >> >> To set custom locale configuration it in necessary to overwrite the >> files of isar-cfg-localepurge before. That can be a bit of a hassle. >> >> With this patch the locale configuration is generated based on bitbake >> variables and therefor can be set from outside. >> >> The new variables are `LOCALE_GEN` which is basically the content of the >> desired `/etc/locale.gen` with each line seperated by '\n' and >> `LOCALE_DEFAULT` which sets the global default locale of the system. >> >> Signed-off-by: Claudius Heine <ch@denx.de> >> --- >> .../isar-cfg-localepurge/files/locale.debconf | 12 ------ >> .../isar-cfg-localepurge/files/locale.gen | 1 - >> .../isar-cfg-localepurge.bb | 43 +++++++++++++++++-- >> 3 files changed, 40 insertions(+), 16 deletions(-) >> delete mode 100644 >> meta/recipes-support/isar-cfg-localepurge/files/locale.debconf >> delete mode 100644 >> meta/recipes-support/isar-cfg-localepurge/files/locale.gen >> >> diff --git >> a/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf >> b/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf >> deleted file mode 100644 >> index 169c071..0000000 >> --- a/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf >> +++ /dev/null >> @@ -1,12 +0,0 @@ >> -locales locales/locales_to_be_generated multiselect >> en_US.UTF-8 UTF-8 >> -locales locales/default_environment_locale select en_US.UTF-8 >> - >> -localepurge localepurge/dontbothernew boolean true >> -localepurge localepurge/nopurge multiselect en, en_US, >> en_US.UTF-8 >> -localepurge localepurge/use-dpkg-feature boolean false >> -localepurge localepurge/verbose boolean false >> -localepurge localepurge/showfreedspace boolean false >> -localepurge localepurge/none_selected boolean false >> -localepurge localepurge/mandelete boolean true >> -localepurge localepurge/quickndirtycalc boolean false >> -localepurge localepurge/remove_no note >> diff --git >> a/meta/recipes-support/isar-cfg-localepurge/files/locale.gen >> b/meta/recipes-support/isar-cfg-localepurge/files/locale.gen >> deleted file mode 100644 >> index a66d814..0000000 >> --- a/meta/recipes-support/isar-cfg-localepurge/files/locale.gen >> +++ /dev/null >> @@ -1 +0,0 @@ >> -en_US.UTF-8 UTF-8 >> diff --git >> a/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb >> b/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb >> index d2cf1f3..f815717 100644 >> --- a/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb >> +++ b/meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb >> @@ -4,12 +4,49 @@ DESCRIPTION = "Isar configuration package for locale >> and localepurge" >> MAINTAINER = "isar-users <isar-users@googlegroups.com>" >> DEBIAN_DEPENDS = "localepurge" >> -SRC_URI = "file://locale.debconf \ >> - file://locale.gen \ >> - file://postinst" >> +SRC_URI = "file://postinst" >> inherit dpkg-raw >> +LOCALE_GEN ?= "en_US.UTF-8 UTF-8\n\ >> + en_US ISO-8859-1\n" >> +LOCALE_DEFAULT ?= "en_US.UTF-8" >> + >> +def get_locale_gen(d): >> + locale_gen = d.getVar("LOCALE_GEN", True) or "" >> + return ', '.join(sorted(set(i.strip() >> + for i in locale_gen.split('\\n')))) >> + >> +def get_nopurge(d): >> + locale_gen = d.getVar("LOCALE_GEN", True) or "" >> + return ', '.join(sorted(set(i.strip() >> + for j in locale_gen.split('\\n') >> + for i in (j.split()[0].split("_")[0], >> + j.split()[0].split(".")[0], >> + j.split()[0])))) >> + >> +do_gen_config[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}" >> +do_gen_config() { >> + cat<<-__EOF__ > ${WORKDIR}/locale.gen >> + ${@(d.getVar("LOCALE_GEN", True) or "").replace('\\n','\n')} >> + __EOF__ >> + cat<<-__EOF__ > ${WORKDIR}/locale.debconf >> + locales locales/locales_to_be_generated multiselect >> ${@get_locale_gen(d)} >> + locales locales/default_environment_locale select >> ${LOCALE_DEFAULT} >> + >> + localepurge localepurge/dontbothernew boolean true >> + localepurge localepurge/nopurge multiselect >> ${@get_nopurge(d)} >> + localepurge localepurge/use-dpkg-feature boolean false >> + localepurge localepurge/verbose boolean false >> + localepurge localepurge/showfreedspace boolean false >> + localepurge localepurge/none_selected boolean false >> + localepurge localepurge/mandelete boolean true >> + localepurge localepurge/quickndirtycalc boolean false >> + localepurge localepurge/remove_no note >> + __EOF__ >> +} >> +addtask gen_config after do_unpack before do_install >> + >> do_install() { >> install -v -d ${D}/usr/lib/${PN} >> install -v -m 644 ${WORKDIR}/locale.debconf \ >> > -- Maxim Osipov ilbers GmbH Maria-Merian-Str. 8 85521 Ottobrunn Germany +49 (151) 6517 6917 mosipov@ilbers.de http://ilbers.de/ Commercial register Munich, HRB 214197 General Manager: Baurzhan Ismagulov ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] meta/isar-cfg-localepurge: move locale configuration inside bb 2018-05-30 8:38 ` Maxim Yu. Osipov @ 2018-05-30 10:32 ` Claudius Heine 0 siblings, 0 replies; 6+ messages in thread From: Claudius Heine @ 2018-05-30 10:32 UTC (permalink / raw) To: Maxim Yu. Osipov, Claudius Heine, isar-users [-- Attachment #1: Type: text/plain, Size: 8460 bytes --] Hi Maxim, On Wed, 2018-05-30 at 10:38 +0200, Maxim Yu. Osipov wrote: > Hi Claudius, > > Yesterday you've sent this patch over the same codebase you've sent > 3 > hours before (One shot configuration patch series V5). > > Is it possible to avoid in the future such frequency of incremental > patches as process of testing/applying patches is rather time > consuming. Yes I will try do do this, but currently not quite sure how. The problem is that I don't like to change a posted and self contained patchset as long as their is no problem with it, so it can be reviewed and eventually merged. Every change to it might delay that. But I still work on top of it and continue adding features and improvements. So I either wait until the patchset is merged before posting or continue posting new patchsets as I work. If I delay and try to have only one patchset on the ML at any time, then I also delay the development time by single threading it. I can do that, but that would be inefficient. I do have the 'release early, release often' mentality when posting patchsets, just to inform people what I am working on and if the might be a conflict or synergy with what others are working on. Hmmm.. maybe I have also a bit different expectations of the review process. When I review patches I would expect that the developer has done some form of testing before posting already and just start reading the code to see what it does. That can be done relatively fast. If I like what it does then I will merge it into a testing branch and test it, afterwards I merge it upstream. For me applying patches and testing them is something done late in the review process. With this patch I wanted the reviewers just to look over the code and comment on it to see if there might be ideas to improve or if this goes into the right direction and not begin with merging and testing testing. Claudius > > Kind regards, > Maxim. > > On 05/30/2018 10:23 AM, Claudius Heine wrote: > > Hi, > > > > sorry, but I messed this patch up. I will fix it an repost it again > > with > > documentation. > > > > Claudius > > > > On 2018-05-29 17:15, [ext] claudius.heine.ext@siemens.com wrote: > > > From: Claudius Heine <ch@denx.de> > > > > > > To set custom locale configuration it in necessary to overwrite > > > the > > > files of isar-cfg-localepurge before. That can be a bit of a > > > hassle. > > > > > > With this patch the locale configuration is generated based on > > > bitbake > > > variables and therefor can be set from outside. > > > > > > The new variables are `LOCALE_GEN` which is basically the content > > > of the > > > desired `/etc/locale.gen` with each line seperated by '\n' and > > > `LOCALE_DEFAULT` which sets the global default locale of the > > > system. > > > > > > Signed-off-by: Claudius Heine <ch@denx.de> > > > --- > > > .../isar-cfg-localepurge/files/locale.debconf | 12 ------ > > > .../isar-cfg-localepurge/files/locale.gen | 1 - > > > .../isar-cfg-localepurge.bb | 43 > > > +++++++++++++++++-- > > > 3 files changed, 40 insertions(+), 16 deletions(-) > > > delete mode 100644 > > > meta/recipes-support/isar-cfg-localepurge/files/locale.debconf > > > delete mode 100644 > > > meta/recipes-support/isar-cfg-localepurge/files/locale.gen > > > > > > diff --git > > > a/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf > > > b/meta/recipes-support/isar-cfg-localepurge/files/locale.debconf > > > deleted file mode 100644 > > > index 169c071..0000000 > > > --- a/meta/recipes-support/isar-cfg- > > > localepurge/files/locale.debconf > > > +++ /dev/null > > > @@ -1,12 +0,0 @@ > > > -locales locales/locales_to_be_generated multiselect > > > en_US.UTF-8 UTF-8 > > > -locales locales/default_environment_locale select > > > en_US.UTF-8 > > > - > > > -localepurge localepurge/dontbothernew boolean true > > > -localepurge localepurge/nopurge multiselect en, en_US, > > > en_US.UTF-8 > > > -localepurge localepurge/use-dpkg-feature boolean false > > > -localepurge localepurge/verbose boolean false > > > -localepurge localepurge/showfreedspace boolean false > > > -localepurge localepurge/none_selected boolean false > > > -localepurge localepurge/mandelete boolean true > > > -localepurge localepurge/quickndirtycalc boolean false > > > -localepurge localepurge/remove_no note > > > diff --git > > > a/meta/recipes-support/isar-cfg-localepurge/files/locale.gen > > > b/meta/recipes-support/isar-cfg-localepurge/files/locale.gen > > > deleted file mode 100644 > > > index a66d814..0000000 > > > --- a/meta/recipes-support/isar-cfg-localepurge/files/locale.gen > > > +++ /dev/null > > > @@ -1 +0,0 @@ > > > -en_US.UTF-8 UTF-8 > > > diff --git > > > a/meta/recipes-support/isar-cfg-localepurge/isar-cfg- > > > localepurge.bb > > > b/meta/recipes-support/isar-cfg-localepurge/isar-cfg- > > > localepurge.bb > > > index d2cf1f3..f815717 100644 > > > --- a/meta/recipes-support/isar-cfg-localepurge/isar-cfg- > > > localepurge.bb > > > +++ b/meta/recipes-support/isar-cfg-localepurge/isar-cfg- > > > localepurge.bb > > > @@ -4,12 +4,49 @@ DESCRIPTION = "Isar configuration package for > > > locale > > > and localepurge" > > > MAINTAINER = "isar-users <isar-users@googlegroups.com>" > > > DEBIAN_DEPENDS = "localepurge" > > > -SRC_URI = "file://locale.debconf \ > > > - file://locale.gen \ > > > - file://postinst" > > > +SRC_URI = "file://postinst" > > > inherit dpkg-raw > > > +LOCALE_GEN ?= "en_US.UTF-8 UTF-8\n\ > > > + en_US ISO-8859-1\n" > > > +LOCALE_DEFAULT ?= "en_US.UTF-8" > > > + > > > +def get_locale_gen(d): > > > + locale_gen = d.getVar("LOCALE_GEN", True) or "" > > > + return ', '.join(sorted(set(i.strip() > > > + for i in > > > locale_gen.split('\\n')))) > > > + > > > +def get_nopurge(d): > > > + locale_gen = d.getVar("LOCALE_GEN", True) or "" > > > + return ', '.join(sorted(set(i.strip() > > > + for j in locale_gen.split('\\n') > > > + for i in > > > (j.split()[0].split("_")[0], > > > + > > > j.split()[0].split(".")[0], > > > + j.split()[0])))) > > > + > > > +do_gen_config[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}" > > > +do_gen_config() { > > > + cat<<-__EOF__ > ${WORKDIR}/locale.gen > > > + ${@(d.getVar("LOCALE_GEN", True) or > > > "").replace('\\n','\n')} > > > + __EOF__ > > > + cat<<-__EOF__ > ${WORKDIR}/locale.debconf > > > + locales locales/locales_to_be_generated > > > multiselect > > > ${@get_locale_gen(d)} > > > + locales locales/default_environment_locale > > > select > > > ${LOCALE_DEFAULT} > > > + > > > + localepurge localepurge/dontbothernew > > > boolean true > > > + localepurge localepurge/nopurge > > > multiselect > > > ${@get_nopurge(d)} > > > + localepurge localepurge/use-dpkg-feature > > > boolean false > > > + localepurge localepurge/verbose > > > boolean false > > > + localepurge localepurge/showfreedspace > > > boolean false > > > + localepurge localepurge/none_selected > > > boolean false > > > + localepurge localepurge/mandelete > > > boolean true > > > + localepurge localepurge/quickndirtycalc > > > boolean false > > > + localepurge localepurge/remove_no note > > > + __EOF__ > > > +} > > > +addtask gen_config after do_unpack before do_install > > > + > > > do_install() { > > > install -v -d ${D}/usr/lib/${PN} > > > install -v -m 644 ${WORKDIR}/locale.debconf \ > > > > > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de PGP key: 6FF2 E59F 00C6 BC28 31D8 64C1 1173 CB19 9808 B153 Keyserver: hkp://pool.sks-keyservers.net [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/1] Improve isar-cfg-localepurge 2018-05-29 15:15 [PATCH 0/1] Improve isar-cfg-localepurge claudius.heine.ext 2018-05-29 15:15 ` [PATCH 1/1] meta/isar-cfg-localepurge: move locale configuration inside bb claudius.heine.ext @ 2018-05-30 11:11 ` Claudius Heine 1 sibling, 0 replies; 6+ messages in thread From: Claudius Heine @ 2018-05-30 11:11 UTC (permalink / raw) To: isar-users; +Cc: Claudius Heine On 2018-05-29 17:15, [ext] claudius.heine.ext@siemens.com wrote: > From: Claudius Heine <ch@denx.de> > > Hi, > > this patchset requires my previous patchset: One-shot configuration packages > > It improves the UX of the isar-cfg-localepurge package and the > customization of the locale. > > It currently does not include a documentation, but that will be added in > a latter version of this patchset. I have decided that I will merge this patchset into the 'One-shot configuration packages'. Claudius > > Cheers, > Claudius > > Claudius Heine (1): > meta/isar-cfg-localepurge: move locale configuration inside bb > > .../isar-cfg-localepurge/files/locale.debconf | 12 ------ > .../isar-cfg-localepurge/files/locale.gen | 1 - > .../isar-cfg-localepurge.bb | 43 +++++++++++++++++-- > 3 files changed, 40 insertions(+), 16 deletions(-) > delete mode 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.debconf > delete mode 100644 meta/recipes-support/isar-cfg-localepurge/files/locale.gen > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-05-30 11:12 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-05-29 15:15 [PATCH 0/1] Improve isar-cfg-localepurge claudius.heine.ext 2018-05-29 15:15 ` [PATCH 1/1] meta/isar-cfg-localepurge: move locale configuration inside bb claudius.heine.ext 2018-05-30 8:23 ` Claudius Heine 2018-05-30 8:38 ` Maxim Yu. Osipov 2018-05-30 10:32 ` Claudius Heine 2018-05-30 11:11 ` [PATCH 0/1] Improve isar-cfg-localepurge Claudius Heine
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox