Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-dev
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-dev <gentoo-dev@g.o>
From: Amadeusz Żołnowski <aidecoe@g.o>
Subject: Re: [RFC] obs eclasses
Date: Tue, 13 Sep 2011 13:24:57 +0200
Hi,


Excerpts from Michal Hrusecky's message of 2011-09-13 13:11:28 +0200:
> Comments and improvements are welcome.

Just some minor remarks:


> [[ -z ${OPENSUSE_RELEASE} ]] || OBS_PROJECT="openSUSE:${OPENSUSE_RELEASE}"
> [[ -n ${OBS_PROJECT} ]]      || die "OBS_PROJECT not set!"
> [[ -n ${OBS_PACKAGE} ]]      || die "OBS_PACKAGE not set!"

You don't need -n/-z with [[.

  [[ $var ]] == [[ -n $var ]]
  [[ ! $var ]] == [[ -z $var ]]

So:

  [[ ${OPENSUSE_RELEASE} ]] && OBS_PROJECT="openSUSE:${OPENSUSE_RELEASE}"
  [[ ${OBS_PROJECT} ]] || die "OBS_PROJECT not set!"
  [[ ${OBS_PACKAGE} ]] || die "OBS_PACKAGE not set!"

> obs-service_src_install() {
>         debug-print-function ${FUNCNAME} "$@"
>         debug-print "Installing service \"${OBS_SERVICE_NAME}\""
>         exeinto /usr/lib/obs/service
>         doexe ${DISTDIR}/${OBS_SERVICE_NAME}
>         insinto /usr/lib/obs/service
>         doins ${DISTDIR}/${OBS_SERVICE_NAME}.service
>         if [[ -n ${ADDITIONAL_FILES} ]]; then
>                 debug-print "Installing following additional files:"
>                 debug-print "        ${ADDITIONAL_FILES}"
>                 exeinto /usr/lib/obs/service/${OBS_SERVICE_NAME}.files
>                 for i in ${ADDITIONAL_FILES}; do
>                         doexe ${DISTDIR}/$i

"" just in case.

>                 done
>         fi
> }

-- 
Amadeusz Żołnowski

PGP key fpr: C700 CEDE 0C18 212E 49DA  4653 F013 4531 E1DB FAB5
Attachment:
signature.asc (PGP signature)
Replies:
Re: [RFC] obs eclasses
-- Joshua Kinard
References:
[RFC] obs eclasses
-- Michal Hrusecky
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
[RFC] obs eclasses
Next by thread:
Re: [RFC] obs eclasses
Previous by date:
[RFC] obs eclasses
Next by date:
Lastrite: xdtv, video-entropyd, qutecom, vdr-pvr350, SDLcam, vdr-avards, vdr-atmo, and gnomeradio


Updated Jun 29, 2012

Summary: Archive of the gentoo-dev mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.