El mar, 01-01-2013 a las 16:01 -0800, Zac Medico escribió:
> On 01/01/2013 05:39 AM, Pacho Ramos wrote:
> > El mar, 01-01-2013 a las 14:32 +0100, Pacho Ramos escribió:
> >> pkg_postinst() {
> >> @@ -48,6 +56,8 @@
> >> elog
> >> fi
> >>
> >> + echo ${CONFIGURATION_INSTRUCTIONS} | fmt | while read -s ELINE; do
> >> elog "${ELINE}"; done
> >> +
> >> # files/systemd/acpid.socket -> ListenStream=/run/acpid.socket
> >> mkdir -p "${ROOT}"/run
> >>
> >> This could probably be moved to eutils.eclass to use it on this kind of
> >> ebuilds
> >
> > Well, "elog" part should be behind:
> > if ! has_version "${CATEGORY}/${PN}"; then
> > echo ${CONFIGURATION_INSTRUCTIONS} | fmt | while read -s ELINE; do elog
> > "${ELINE}"; done
> > fi
> >
>
> Not that `has_version "${CATEGORY}/${PN}"` is always true in
> pkg_postinst, since the package is already installed. So, you should
> choose one of these alternatives for it to work as intended:
>
> 1) call has_version in pkg_preinst
> 2) use [[ ${REPLACING_VERSIONS} ]] instead
> Yeah, that is true (and looks like current acpid ebuild is buggy in this). I wouldn't have any problem on either solution, but using first one would work in all eapis, is there any reason for printing this kind of messages in pkg_postinst?
signature.asc
Description: This is a digitally signed message part
