[Bug 195363] Review Request: esc and esc-xulrunner-devel
bugzilla at redhat.com
bugzilla at redhat.com
Fri Jul 14 14:31:59 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: esc and esc-xulrunner-devel
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=195363
rstrode at redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |NEEDINFO
Flag| |needinfo?(jmagne at redhat.com)
------- Additional Comments From rstrode at redhat.com 2006-07-14 10:23 EST -------
So the esc-xulrunner-devel package is sort of weird.
First, we don't normally put -devel in the name of srpms. -devel is usually
reserved for subpackages.
Also, you install all header files, images, config files, stylesheets, binaries,
etc into %{_libdir}. Normally we put header files in %{_includedir}, images in
%{_datadir}, binaries in %{_bindir}, etc.
On the other hand, esc-xulrunner-devel is only needed for building esc, yea? not
for running esc? Why are we installing it as a separate package at all then?
Can we just put the xulrunner tarball as an extra Source: line in the spec file
and get rid of esc-xulrunner-devel entirely?
Also, I tried to build the two packages and ran into problems. esc looks for
something called nsinstall in wrong place. I had to create a symlink for the
build to finish.
After I got it built, it didn't work with our cert server. It gave me an error
code 44 or something. This actually brings up another point.
This tool only works with a closed source certificate server that most in the
fedora community don't have access to. Maybe it would be better if we put this
in extras instead of core? What do you think?
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
More information about the Fedora-package-review
mailing list