Jan Harkes writes:
On Tue, Mar 28, 2006 at 11:20:56PM -0600, Jerry Amundson wrote:
Alright, I can handle those easily enough, but not this one... g++ -fno-exceptions -fcheck-new -Wall -MD -DHAVE_CONFIG_H -I. -I/home/jerry/rpm/BUILD/coda-6.0.14/coda-src/venus -I/home/jerry/rpm/BUILD/obj-6.0.14/include -I/home/jerry/rpm/BUILD/obj-6.0.14 -g -O2 -DVENUS -DTIMING -DVENUSDEBUG -DRVM_USELWP -c -o vsg.o /home/jerry/rpm/BUILD/coda-6.0.14/coda-src/venus/vsg.cc /home/jerry/rpm/BUILD/coda-6.0.14/coda-src/venus/vsg.cc: In member function ‘int vsgent::GetMgrp(mgrpent**, uid_t, int)’: /home/jerry/rpm/BUILD/coda-6.0.14/coda-src/venus/vsg.cc:105: error: ‘Mgrp_Wait’ was not declared in this scope
make: *** [vsg.o] Error 1

I haven't had a chance to look at these yet, but someone just sent a
while batch of patches that fix these FC5 compilation problems. http://www.coda.cs.cmu.edu/rt/Ticket/Display.html?id=1529
Jan

Just posted this in RT, but the Result window showed "Message recorded
Ticket 1529: Permission Denied" (even though it displayed immediately on the page), so copying here also... jerry
---- RT posting ----
The spec files should not assume packaging as root, a security no-no. The lines
BuildRoot: /usr/src/redhat/BUILD/@[EMAIL PROTECTED]@VERSION@/Root
are also non-portable : Suse packages, for example will fail. Better to use something
like
BuildRoot: %{_builddir}/@[EMAIL PROTECTED]@VERSION@/Root
or, ideally if it's possible
BuildRoot: %{_builddir}/%{name}-%{version}/Root
The latter preferred, as it avoids the chicken-egg problem that the spec needs configure be run first, but rpm builds are designed to run configure for you. Finally, the
coda.spec.in
should use that format to be consistent with the others.

I am also working on spec file improvement. Next after the above are BuildRequires, and Requires, to accurately reflect dependencies between the four modules. I'll upload those
patches when ready - possibly a day or so.

Reply via email to