[Bug 227309] Review Request: seom - Desktop video capture and playback utility

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: seom - Desktop video capture and playback utility


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227309





------- Additional Comments From tom@xxxxxxxxxxxxx  2007-02-08 12:54 EST -------
(In reply to comment #13)
> I think we could probably use something along the lines of a README that
> includes a bit stating that you're the author of the bulk of the code, with a
> reference stating where the codec.c bit came from and that it is also GPL, just
> to cover all the bases.
> 

will do ... I'll drop you an email as soon as a new tarball is available.

> For Fedora packages, libtool archives and static libs are forbidden, so it was
> solely for the benefit of the package. I'll use a patch that doesn't nuke that,
> and will instead remove the file from within the spec.

I would actually love to drop libtool, but it's a nice tool that adds '-fPIC'
when needed (and other arch-dependent flags). But other than that, I find it
horrible (what are these libtool archives for anyway? The compiler doesn't need
them when linking against the library, right?).

> 
> Other notes on the patch I'm using:
> - splitting CFLAGS and EXTRA_CFLAGS makes packaging much easier. Fedora has a
> standard set of CFLAGS that are supposed to be used on all packages, so
> splitting off the -std=c99 bit into EXTRA_CFLAGS makes life easier, since its
> required, but not part of the standard Fedora CFLAGS.

What about 'CFLAGS += ...' in the Makefile and a configure option --cflags,
would that work for you?

> - some of the added $(*DIR) bits are for convenience more than anything, but are
> fairly standard.

One thing that still bothers be is my use of LIBDIR. I know it's non-standard,
but I find it much more convenient when cross-compiling, because I only have to
set LIBDIR='lib' or 'lib32' and not LIBDIR='$PREFIX/lib'. I also don't really
see a use in installing the libs, apps and headers each into a different prefix.
I'm hard to convince, I admit, but my decisions are rarely set in stone, so I'm
sure if you give me strong, good reasons I may change my use of LIBDIR ;)

> - you've got lots of extraneous slashes -- $(DESTDIR)/$(PREFIX) works out to
> /somewhere//usr -- so I nuke all those.

Yeah, I noticed that, too, but i don't care because it's just a cosmetic change,
or isn't it?


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]