Re: [OE-core] [PATCH v2 0/3] postinst fixes for opgk/dpkg

2018-06-14 Thread Alexander Kanavin
No need to be nervous. The patch will make it in; patches do not get
quietly rejected. If it hasn't showed up yet, it's most likely because
the patch will be tested on the autobuilder in a separate set from the
'regular patches', or (yes, this happens) it has slipped through the
cracks. In that case (I know it's impossible to tell from the
outside), just resend again.

Also, please do help us out with oe-core patch review and recipe
maintenance, particularly updating them to latest versions.


Alex

2018-06-14 16:57 GMT+03:00 Stefan Agner :
> On 04.06.2018 18:18, Alexander Kanavin wrote:
>> On 06/04/2018 07:06 PM, Stefan Agner wrote:
>>
>>> Anything holding us back merging this changes?
>>
>> Please read the Yocto project status mail:
>>
>> " · Patch review/merging was slow over the past week due to
>> Ross being on vacation and changes in Richard’s employment/hardware
>> situation but at least some of the backlog was resolved over the
>> weekend and the hardware issues should be worked around now."
>>
>
> Gentle ping on that again.
>
> This week it read:
> "Due to the milestone, patch merging is slowing to stabilize."
>
> Also, there have been patches merged which were posted after this
> patchset... It is really rather important for us since it is hard to fix
> in lower layers...
>
> So I am getting nervous whether it still makes it?
>
> --
> Stefan
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v2 0/3] postinst fixes for opgk/dpkg

2018-06-14 Thread Stefan Agner
On 04.06.2018 18:18, Alexander Kanavin wrote:
> On 06/04/2018 07:06 PM, Stefan Agner wrote:
> 
>> Anything holding us back merging this changes?
> 
> Please read the Yocto project status mail:
> 
> " · Patch review/merging was slow over the past week due to
> Ross being on vacation and changes in Richard’s employment/hardware
> situation but at least some of the backlog was resolved over the
> weekend and the hardware issues should be worked around now."
> 

Gentle ping on that again.

This week it read:
"Due to the milestone, patch merging is slowing to stabilize."

Also, there have been patches merged which were posted after this
patchset... It is really rather important for us since it is hard to fix
in lower layers...

So I am getting nervous whether it still makes it?

--
Stefan
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v2 0/3] postinst fixes for opgk/dpkg

2018-06-04 Thread Alexander Kanavin

On 06/04/2018 07:06 PM, Stefan Agner wrote:


Anything holding us back merging this changes?


Please read the Yocto project status mail:

" · Patch review/merging was slow over the past week due to Ross 
being on vacation and changes in Richard’s employment/hardware situation 
but at least some of the backlog was resolved over the weekend and the 
hardware issues should be worked around now."


Alex
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH v2 0/3] postinst fixes for opgk/dpkg

2018-06-04 Thread Stefan Agner
On 16.05.2018 11:13, Stefan Agner wrote:
> From: Stefan Agner 
> 
> Hi,
> 
> This follows up on the discussion a while ago:
> https://www.mail-archive.com/openembedded-core@lists.openembedded.org/msg104996.html
> 
> Patch 1 is rather simple and really fixes the main issue. The patch
> by itself has been tested with the relevant self test and passes.
> 
> Patch 2/3 get rid of /etc/*-postinsts script in cases where the package
> management is present. This avoids a bunch of unnecessary scripts to be
> present on the rootfs. It also avoids the systemd service to be present
> forever with in case no postinst scripts have been deployed:
> Condition: start condition failed at Tue 2018-05-15 10:57:43 UTC; 42s ago
>└─ ConditionPathExistsGlob=/etc/*-postinsts was not met
> 
> Self test executed using:
> $ oe-selftest --run-tests runtime_test.Postinst.test_postinst_rootfs_and_boot

Anything holding us back merging this changes?

--
Stefan

> 
> Changes since v1:
> - Note that patches are specific for opkg/dpkg
> 
> Stefan Agner (3):
>   opkg: avoid running postinst scripts twice when using systemd
>   run-postinsts: for dpkg/opkg, do not rely on /etc/*-postinsts
>   rootfs.py: for dpkg/opkg, don't install postinsts if package
> management is present
> 
>  meta/lib/oe/rootfs.py   |  3 +++
>  .../opkg/opkg/opkg-configure.service| 17 -
>  meta/recipes-devtools/opkg/opkg_0.3.6.bb| 14 --
>  .../run-postinsts/run-postinsts/run-postinsts   | 21 
> -
>  .../run-postinsts/run-postinsts.service |  1 -
>  5 files changed, 15 insertions(+), 41 deletions(-)
>  delete mode 100644 meta/recipes-devtools/opkg/opkg/opkg-configure.service
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core