2008/9/30 Miklos Vajna <[EMAIL PROTECTED]>

> On Mon, Sep 29, 2008 at 08:34:27PM +0200, Devil505 <
> [EMAIL PROTECTED]> wrote:
> > I'm trying to make a package for emul32-tools. I'm not really sure about
> > depends() and group(), maybe it's apps-extra...
>
> I think all those emul pkgs depends on glibc-emul, so that's not
> necesary. Also, frugalware-emul could depend on coreutils-emul, I don't
> think all packages using commands like 'mv', 'rm', etc. has coreutils in
> depends.
>
> The group is fine IMHO.


OK I keep emul-extra


>
>
> > By the way, I fixed the Readme file ("repoman merge teamspeak-client").
> > I installed the package and "emul32 teamspeak" command works :)
>
> Nice!
>
> > url="http://frugalware.org/";
>
> or http://ftp.frugalware.org/pub/other/$pkgname?
>

fixed


>
> > archs=('!i686' 'x86_64')
>
> Not that matters much, but you can add !ppc as well. :)
>

fixed


>
> > up2date="Flasttar http://ftp.frugalware.org/pub/other/sources/$pkgname/";
> > source=(
> http://ftp.frugalware.org/pub/other/sources/$pkgname/$pkgname-$pkgver.tar.bz2README.Frugalware)
>
> Given taht we use /pub/other/pkgname for our packages and
> /pub/other/sources for packages with broken upstream, I think this is
> not so good.
>

fixed


>
> > build()
> > {
> >     Fcd
> >     Fexerel emul32 usr/bin/emul32
> >     Fexerel rc.$pkgname /etc/rc.d/rc.$pkgname
> >     Fdoc README.Frugalware
> > }
>
> Fine - or you can just write a simple Makefile that does this for you
> and just omit build() at all. See 'git show 63c4e:Makefile' in
> pacman-tools.git for example.
>

fixed with your help this afternoon


>
> > ----
> > krics ALL=(ALL) NOPASSWD: /usr/bin/emul32
> > ----
>
> It's not you who wrote this line, but maybe use something like <user>
> here?


fixed


>
>
> The rest looks fine.
>

May have to push this package ? :)

Daniel

Attachment: FrugalBuild
Description: Binary data

_______________________________________________
Frugalware-devel mailing list
[email protected]
http://frugalware.org/mailman/listinfo/frugalware-devel

Reply via email to