[Bug 192049] Review Request: gnash - GNU Flash player

bugzilla at redhat.com bugzilla at redhat.com
Wed May 17 14:51:47 UTC 2006


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: gnash - GNU Flash player


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





------- Additional Comments From pertusus at free.fr  2006-05-17 10:51 EST -------
(In reply to comment #3)
 
> > * there is a security issue that should be patched in fedora extras package, 
> >   indeed there is an insecure use of /tmp. If it is too much work, at least
> >   there should be a note somewhere.
> 
> Is there a patch from cvs that can be backported for this?

Unfortunately not. I raised the issue on the gnash-dev list, though.
That's the only thing that retained me from proposing gnash to FE ;-)
 
> > * the documentation should be distributed (see the specfile in the tarball
> >   for hints on how to do this), except if there is a good reason not to 
> >   distribute it? At least manpage and html manual, info files and 
> >   scrollkeeper files would be bonus 
> 
> Sounds good.  I added buildrequires docbook2X for that.

This shouldn't be needed, as the files are allready up to date in the
tarball. Doesn't hurt, though.

There is a bug in doc/C/Makefile.am regarding info files installation
during staged install. I'll attach patches.

> > W: gnash devel-file-in-non-devel-package /usr/lib/libgnashbase.so
> 
> I removed them for now.

Agreed, this seems to be the best solution until they are really usefull
to develop something.

> I also subpackaged the libraries.

Really? It doesn't seems so to me... Anyway I think it is better to have
the libraries together with the standalone player. But if you want to
split, you can;


-- 
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.




More information about the Fedora-package-review mailing list