Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Andreas Oberritter <obi@opendreambox.org>
To: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH] avahi-systemd: drop postrm, use prerm instead
Date: Mon, 21 May 2012 13:49:01 +0200	[thread overview]
Message-ID: <4FBA2BAD.5070709@opendreambox.org> (raw)
In-Reply-To: <lyd35xeqw9.fsf@ensc-virt.intern.sigma-chemnitz.de>

On 21.05.2012 13:04, Enrico Scholz wrote:
> Andreas Oberritter <obi@opendreambox.org> writes:
> 
>>>>>> What's the use case for removing packages offline
>>>
>>> yes; I am working on the build machine. The NFS filesystem is mounted
>>> read-only on the target.
>>
>> How do you handle prerm and postrm scripts that fail because $D is
>> set?
> 
> Fortunately, there are very few packages where this failure is
> critical (--> package is not working).  Important tasks like user
> creation, alternatives handling or systemctl calls take care about
> offline package management already.
> 
> kernel modules are the only package class which might affect me and is
> not suited for offline installation/upgrades.  But I can live with it
> because I develop the kernel at a separate location and call 'make
> modules_install INSTALL_MOD_PATH=<nfsroot>' there.
> 
> 
>> How do you handle the majority of scripts that don't care about $D at
>> all?
> 
> Examples?

Every recipe inheriting gconf.bbclass, gtk-icon-cache.bbclass,
kernel.bbclass, module.bbclass or libc-package.bbclass, for example. You
can use git grep '$D' to find more candidates.

>  pseudo does a good job for 'systemctl enable/disable' or
> update-alternatives operations, 'systemd.bbclass' wraps the 'systemctl
> stop/start'.
> 
> Btw... your patch is correct about removal of $D. As written above, it
> is not needed for 'systemctl disable'.  But I am against removal of
> offline capabilities because they are not needed in 80% of use cases.

Considering that in the meantime my similar patch [1] to systemd.bbclass
got merged into meta-oe by Koen, who initially was against it, I guess
this patch can finally get merged now, too.

Regards,
Andreas

[1]
http://git.openembedded.org/meta-openembedded/commit/?id=637cb7e3d2cfdc74d239a4257e6f3477aa17da4e



  reply	other threads:[~2012-05-21 11:59 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-16 16:55 [PATCH] avahi-systemd: drop postrm, use prerm instead Andreas Oberritter
2012-05-16 17:17 ` Koen Kooi
2012-05-16 18:38   ` Andreas Oberritter
2012-05-16 18:50     ` Koen Kooi
2012-05-16 19:03       ` Andreas Oberritter
2012-05-18 15:06     ` Enrico Scholz
2012-05-21  8:46       ` Andreas Oberritter
2012-05-21  9:18         ` Enrico Scholz
2012-05-21 10:08           ` Andreas Oberritter
2012-05-21 11:04             ` Enrico Scholz
2012-05-21 11:49               ` Andreas Oberritter [this message]
2012-05-21 12:06                 ` Koen Kooi
2012-05-21 12:13                   ` Enrico Scholz
2012-05-21 12:20                     ` Koen Kooi
2012-05-21 12:33                       ` Enrico Scholz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FBA2BAD.5070709@opendreambox.org \
    --to=obi@opendreambox.org \
    --cc=enrico.scholz@sigma-chemnitz.de \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox