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 > > > > > > 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 > > > --- > > > .../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 " > > > 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