Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in www-servers/resin: ChangeLog resin-3.0.25.ebuild resin-3.1.4.ebuild

2007-12-16 Thread Bo Ørsted Andresen
On Sunday 16 December 2007 08:03:37 Donnie Berkholz wrote:
> > chmod 755 "${S}/configure"
>
> fperms means you don't need S or quotes.

fperms works on D. Not S.

-- 
Bo Andresen


signature.asc
Description: This is a digitally signed message part.


Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in media-video/undvd: undvd-0.3.0.ebuild metadata.xml ChangeLog Manifest

2007-12-16 Thread Bo Ørsted Andresen
On Sunday 16 December 2007 08:13:49 Donnie Berkholz wrote:
> On 17:46 Fri 14 Dec , Greg Kroah-Hartman (gregkh) wrote:
> > pkg_setup() {
> > einfo "Checking mplayer for USE flags we need..."
> > for f in "encode dvd x264 mp3"; do
> > if ! built_with_use media-video/mplayer $f; then
> > eerror "$f"
> > die "mplayer merged without $f USE flag"
> > fi
> > done
>
> Does this actually work as you want? I'd be surprised. Quoting the whole
> thing should treat it as a single flag with spaces and only run through
> the loop once.
>
> It should only coincidentally work because built_with_use() takes any
> number of flags and loops through them, but I doubt it prints the single
> missing flag as you intend.

Indeed it works because $f isn't quoted and built_with_use handles a list just 
fine. Just the for loop is pointless. Fixing it to check only one use flag at 
a time, however, is a horrible idea because it risks telling the user to 
recompile mplayer four times each adding another use flag. There really is no 
reason not to just show the full list of required flags if one of them is 
missing but if you really want to at least move the die out of the loop.

-- 
Bo Andresen


signature.asc
Description: This is a digitally signed message part.


Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in www-servers/resin: ChangeLog resin-3.0.25.ebuild resin-3.1.4.ebuild

2007-12-16 Thread Donnie Berkholz
On 12:40 Sun 16 Dec , Bo Ørsted Andresen wrote:
> On Sunday 16 December 2007 08:03:37 Donnie Berkholz wrote:
> > >   chmod 755 "${S}/configure"
> >
> > fperms means you don't need S or quotes.
> 
> fperms works on D. Not S.

Good point. It's worth noting that this was the only file in S, and the 
others are all in D.

Thanks,
Donnie
-- 
[EMAIL PROTECTED] mailing list



Re: [gentoo-dev] X drivers up for grabs

2007-12-16 Thread Mike Frysinger
On Monday 03 December 2007, Donnie Berkholz wrote:
> I need to lower my maintenance burden a bit, and it makes sense to do
> this by handing off things I don't have hardware for anyway.

you didnt list it, but i can help with the sisusb driver ... i have one of 
those neat dongles
-mike


signature.asc
Description: This is a digitally signed message part.


[gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in app-cdr/tkdvd: ChangeLog tkdvd-3.10.1.ebuild tkdvd-4.0.4.ebuild tkdvd-4.0.7.ebuild

2007-12-16 Thread Donnie Berkholz
On 14:50 Sun 16 Dec , Lars Weiler (pylon) wrote:
> 1.1  app-cdr/tkdvd/tkdvd-4.0.7.ebuild
> 
> file : 
> http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-cdr/tkdvd/tkdvd-4.0.7.ebuild?rev=1.1&view=markup
> plain: 
> http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-cdr/tkdvd/tkdvd-4.0.7.ebuild?rev=1.1&content-type=text/plain

> src_install() {
>   insinto /usr/share/${PF}/src
>   doins src/*
> 
>   exeinto /usr/share/${PF}
>   doexe TkDVD.sh
> 
>   dodir /usr/bin
> 
>   cat <"${D}"/usr/bin/tkdvd
> #!/bin/sh
> cd /usr/share/${PF}
> ./TkDVD.sh
> EOF
> 
>   fperms 755 /usr/bin/tkdvd
> 
>   dodoc ChangeLog FAQ INSTALL README TODO doc/config_file
> 
>   insinto /usr/share/pixmaps
>   doins icons/*.png
> }

This is a nice counterexample of what Mike wrote the other day about 
maintaining indentation when using here documents. If the '<<' changed 
to '<<-', you could indent the script to keep the ebuild more readable.

Thanks,
Donnie
-- 
[EMAIL PROTECTED] mailing list



[gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in media-video/undvd: undvd-0.3.0.ebuild metadata.xml ChangeLog Manifest

2007-12-16 Thread Duncan
Donnie Berkholz <[EMAIL PROTECTED]> posted
[EMAIL PROTECTED], excerpted below, on  Sun, 16 Dec 2007
12:49:13 -0800:

> I think it's valuable to show the flags that actually need to be changed
> rather than a full list of all required flags.

++

I had a USE blocker give me a list, recently, and it was mildly 
irritating to have to sort out which of the several USE flags were wrong.

In my case it was in kdelibs-.4 in the KDE overlay, complaining about 
qt4, recent changes according to the forum, but it still took me a bit to 
figure out which USE flag it was, even tho I had read about them, because 
there were several flags in the list, all of which seemed set on first 
glance -- but I missed accessibility.  So even knowing about it in 
advance, the list wasn't as clear as naming the specific flag would have 
been.

Still, I agree a list is better than having to recompile it several times 
over! =8^)

-- 
Duncan - List replies preferred.   No HTML msgs.
"Every nonfree program has a lord, a master --
and if you use the program, he is your master."  Richard Stallman

-- 
[EMAIL PROTECTED] mailing list



Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in media-video/undvd: undvd-0.3.0.ebuild metadata.xml ChangeLog Manifest

2007-12-16 Thread Bo Ørsted Andresen
On Monday 17 December 2007 00:48:12 Duncan wrote:
> > I think it's valuable to show the flags that actually need to be changed
> > rather than a full list of all required flags.
>
> ++
>
> I had a USE blocker give me a list, recently, and it was mildly
> irritating to have to sort out which of the several USE flags were wrong.

You don't need to know. Just put all of them in package.use.

-- 
Bo Andresen


signature.asc
Description: This is a digitally signed message part.


[gentoo-dev] Automated Package Removal and Addition Tracker, for the week ending 2007-12-16 23h59 UTC

2007-12-16 Thread Robin H. Johnson
The attached list notes all of the packages that were added or removed
from the tree, for the week ending 2007-12-16 23h59 UTC.

Removals:
app-text/cstetex2007-12-11 21:10:02 aballier
app-emacs/df2007-12-14 08:13:15 ulm

Additions:
sci-biology/mammoth 2007-12-10 09:11:28 dberkholz
media-libs/slv2 2007-12-10 21:41:52 aballier
media-plugins/swh-lv2   2007-12-10 21:52:56 aballier
dev-java/hoteqn 2007-12-10 22:37:44 caster
sci-chemistry/chemical-mime-data2007-12-11 04:36:01 je_fro
sci-chemistry/gchemutils2007-12-11 04:50:25 je_fro
sci-chemistry/gchempaint2007-12-11 05:00:23 je_fro
games-puzzle/candycrisis2007-12-11 21:17:22 tupone
media-plugins/vdr-noepgmenu 2007-12-11 22:13:13 zzam
media-plugins/vdr-exec  2007-12-11 22:51:22 zzam
sys-apps/spu-tools  2007-12-12 01:33:18 lu_zero
media-plugins/amb-plugins   2007-12-13 11:24:09 aballier
media-sound/vdramgw 2007-12-13 13:20:29 zzam
media-plugins/vdr-amarok2007-12-13 13:21:16 zzam
dev-util/d-feet 2007-12-13 20:41:12 cardoe
app-emacs/df-mode   2007-12-14 08:03:58 ulm
dev-util/qdevelop   2007-12-14 12:47:35 jokey
media-video/undvd   2007-12-14 17:46:51 gregkh
dev-haskell/binary  2007-12-15 18:37:00 dcoutts
dev-haskell/bzlib   2007-12-15 18:37:41 dcoutts
dev-haskell/iconv   2007-12-15 18:38:13 dcoutts
dev-haskell/zlib2007-12-15 18:38:45 dcoutts
dev-java/pat2007-12-15 20:25:23 betelgeuse
dev-ruby/open4  2007-12-15 22:07:45 nichoj
dev-ruby/vlad   2007-12-15 22:22:52 nichoj
x11-wm/xmonad   2007-12-15 22:58:29 kolmodin
x11-wm/xmonad-contrib   2007-12-15 23:23:57 kolmodin
x11-misc/xmobar 2007-12-15 23:28:17 kolmodin
kde-base/knetwalk   2007-12-16 18:26:51 philantrop
gnome-extra/swfdec-gnome2007-12-16 18:52:39 pclouds
media-radio/svxlink 2007-12-16 19:25:39 pylon

--
Robin Hugh Johnson
Gentoo Linux Developer
E-Mail : [EMAIL PROTECTED]
GnuPG FP   : 11AC BA4F 4778 E3F6 E4ED  F38E B27B 944E 3488 4E85
Removed Packages:
app-text/cstetex,removed,aballier,2007-12-11 21:10:02
app-emacs/df,removed,ulm,2007-12-14 08:13:15
Added Packages:
sci-biology/mammoth,added,dberkholz,2007-12-10 09:11:28
media-libs/slv2,added,aballier,2007-12-10 21:41:52
media-plugins/swh-lv2,added,aballier,2007-12-10 21:52:56
dev-java/hoteqn,added,caster,2007-12-10 22:37:44
sci-chemistry/chemical-mime-data,added,je_fro,2007-12-11 04:36:01
sci-chemistry/gchemutils,added,je_fro,2007-12-11 04:50:25
sci-chemistry/gchempaint,added,je_fro,2007-12-11 05:00:23
games-puzzle/candycrisis,added,tupone,2007-12-11 21:17:22
media-plugins/vdr-noepgmenu,added,zzam,2007-12-11 22:13:13
media-plugins/vdr-exec,added,zzam,2007-12-11 22:51:22
sys-apps/spu-tools,added,lu_zero,2007-12-12 01:33:18
media-plugins/amb-plugins,added,aballier,2007-12-13 11:24:09
media-sound/vdramgw,added,zzam,2007-12-13 13:20:29
media-plugins/vdr-amarok,added,zzam,2007-12-13 13:21:16
dev-util/d-feet,added,cardoe,2007-12-13 20:41:12
app-emacs/df-mode,added,ulm,2007-12-14 08:03:58
dev-util/qdevelop,added,jokey,2007-12-14 12:47:35
media-video/undvd,added,gregkh,2007-12-14 17:46:51
dev-haskell/binary,added,dcoutts,2007-12-15 18:37:00
dev-haskell/bzlib,added,dcoutts,2007-12-15 18:37:41
dev-haskell/iconv,added,dcoutts,2007-12-15 18:38:13
dev-haskell/zlib,added,dcoutts,2007-12-15 18:38:45
dev-java/pat,added,betelgeuse,2007-12-15 20:25:23
dev-ruby/open4,added,nichoj,2007-12-15 22:07:45
dev-ruby/vlad,added,nichoj,2007-12-15 22:22:52
x11-wm/xmonad,added,kolmodin,2007-12-15 22:58:29
x11-wm/xmonad-contrib,added,kolmodin,2007-12-15 23:23:57
x11-misc/xmobar,added,kolmodin,2007-12-15 23:28:17
kde-base/knetwalk,added,philantrop,2007-12-16 18:26:51
gnome-extra/swfdec-gnome,added,pclouds,2007-12-16 18:52:39
media-radio/svxlink,added,pylon,2007-12-16 19:25:39

Done.

[gentoo-dev] New-style virtuals LICENSE variable

2007-12-16 Thread Mark Loeser
Just to bring to a wider audience the discussion that we had on bug
#140180 [1].

While documenting new-style virtuals for devmanual we began to discuss
what the LICENSE variable for the virtual ebuilds should contain.  For
the reasons listed on the bug, we came to a conclusion that the LICENSE
variable should be empty for these ebuilds.

A brief list of the reasons:

* the ebuild doesn't really install anything
* the ebuild itself is already licensed under GPLv2 as is every
  other ebuild in the tree
* ACCEPT_LICENSE accepts an empty LICENSE, so the virtual is
  automatically accepted
* if the above were not true, we would have the fun maintainence
  nightmare of having to list every license that could satisfy the
  virtual in the LICENSE variable

So long as everyone understands this, I'll go ahead and commit the new
documentation to devmanual and ask zmedico to commit the repoman change
to make sure LICENSE="" in the virtual category.

Thanks,


[1] https://bugs.gentoo.org/show_bug.cgi?id=140180
-- 
Mark Loeser
email -   halcy0n AT gentoo DOT org
web   -   http://www.halcy0n.com


pgpEHiB1YmrR7.pgp
Description: PGP signature


[gentoo-dev] New-style virtuals LICENSE variable

2007-12-16 Thread Mark Loeser
(Sorry for the spam g-dev, I forgot to send this to dev-announce as
well).

- Forwarded message from Mark Loeser <[EMAIL PROTECTED]> -

> Date: Sun, 16 Dec 2007 20:03:39 -0500
> From: Mark Loeser <[EMAIL PROTECTED]>
> To: gentoo-dev@lists.gentoo.org
> Subject: [gentoo-dev] New-style virtuals LICENSE variable
> User-Agent: Mutt/1.5.16 (2007-06-09)
> 
> Just to bring to a wider audience the discussion that we had on bug
> #140180 [1].
> 
> While documenting new-style virtuals for devmanual we began to discuss
> what the LICENSE variable for the virtual ebuilds should contain.  For
> the reasons listed on the bug, we came to a conclusion that the LICENSE
> variable should be empty for these ebuilds.
> 
> A brief list of the reasons:
> 
> * the ebuild doesn't really install anything
> * the ebuild itself is already licensed under GPLv2 as is every
>   other ebuild in the tree
> * ACCEPT_LICENSE accepts an empty LICENSE, so the virtual is
>   automatically accepted
> * if the above were not true, we would have the fun maintainence
>   nightmare of having to list every license that could satisfy the
>   virtual in the LICENSE variable
> 
> So long as everyone understands this, I'll go ahead and commit the new
> documentation to devmanual and ask zmedico to commit the repoman change
> to make sure LICENSE="" in the virtual category.
> 
> Thanks,
> 
> 
> [1] https://bugs.gentoo.org/show_bug.cgi?id=140180
> -- 
> Mark Loeser
> email -   halcy0n AT gentoo DOT org
> web   -   http://www.halcy0n.com



- End forwarded message -


pgpWnxw422NtR.pgp
Description: PGP signature


Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in www-servers/resin: ChangeLog resin-3.0.25.ebuild resin-3.1.4.ebuild

2007-12-16 Thread Krzysiek Pawlik
Donnie Berkholz wrote:
>> 1.1  www-servers/resin/resin-3.1.4.ebuild
>> src_unpack() {
>>
>>  unpack ${A}
>>  for i in "${WORKDIR}"/${PV}/resin-${PV}-*; do
>>  epatch "${i}"
>>  done;
> 
> How about:
> 
> epatch "${WORKDIR}"/${PV}/resin-${PV}-*

Leftover from tests, will be fixed with next bump.

>> src_compile() {
>>
>>  append-flags -fPIC -DPIC
> 
> Somehow it seems wrong to append this for a whole package...

It builds only one .so library - the rest is in Java, so I don't see a problem
with using append-flags.

>> src_install() {
>>
>>  make DESTDIR="${D}" install || die
> 
> Does emake work? If not, please add a comment to that effect.

Didn't test, but I will with next bump.

>>  einfo "Fixing permissions..."
>>  chown -R resin:resin "${D}${RESIN_HOME}"
>>  chown -R resin:resin "${D}/etc/resin"
>>  chown -R resin:resin "${D}/var/log/resin"
>>  chown -R resin:resin "${D}/var/lib/resin"
>>  chown -R resin:resin "${D}/var/run/resin"
>>
>>  chmod 644 "${D}/etc/conf.d/resin"
>>  chmod 755 "${D}/etc/init.d/resin"
>>  chmod 750 "${D}/var/lib/resin"
>>  chmod 750 "${D}/var/run/resin"
>>  chmod 750 "${D}/etc/resin"
> 
> Again, fowners/fperms can make this a bit easier to read.

I'll fix it with next bump.

-- 
Krzysiek Pawlik  key id: 0xBC51
desktop-misc, java, apache, ppc, vim, kernel...



signature.asc
Description: OpenPGP digital signature