Red Hat Bugzilla – Bug 179707
Review Request: dap-server - Basic request handling for DAP servers
Last modified: 2007-11-30 17:11:22 EST
SRPM Name or Url: http://www.environnement.ens.fr/perso/dumas/fc-srpms/dap-server-3.5.3-1.src.rpm
This is base software for our workhorse server. Written using the DAP++ C++
library and Perl, this handles processing compressed files and arranging for
the correct server module to process the file. The base software also
provides support for the ASCII response and HTML data-request form. Use this
in combination with one or more of the format-specific handlers.
This package contains all the executable and perl modules. The scripts
and config files that should be installed in a cgi directory are in the
documentation directory with -sample appended. If you want a package that
is usable without manual configuration, install dap-server-cgi.
Hi Patrice, I've been meaning to look at this so, since today is the first
scheduled FE "review day", heres a quick review:
+ source matches upstream
+ license is correctly included
+ spec is very legible--no obvious errors
+ builds on FC4 (not in mock)
+ rpmlint reports:
E: dap-server non-standard-uid /var/cache/dap-server apache
E: dap-server non-standard-gid /var/cache/dap-server apache
W: dap-server-cgi no-documentation
E: dap-server-cgi non-standard-uid \
...and a few more of the same apache uid/gid errors...
which can probably be safely ignored
+ dir ownership is OK
+ no shared libs
+ clean(s) are properly done
- License is LGPL not GPL
- fails to build with "mock -r fedora-development-i386-core $SRPM"
and returns the following error message:
make: Entering directory `/builddir/build/BUILD/dap-server-3.5.3'
if g++ -DHAVE_CONFIG_H -I. -I. -I. -I/usr/include/libdap -O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4
-m32 -march=i386 -mtune=generic -fasynchronous-unwind-tables -MT usage.o -MD -MP
-MF ".deps/usage.Tpo" -c -o usage.o usage.cc; \
then mv -f ".deps/usage.Tpo" ".deps/usage.Po"; else rm -f ".deps/usage.Tpo";
exit 1; fi
/usr/include/libdap/GNURegex.h:45: error: extra qualification 'Regex::' on
Do you know whats the matter with devel at the moment? I realize its
not a blocker, I'm just curious whats causing the problem -- and don't
have a lot of time right now to look at the syntax...
It is certainly a non conformant construct revealed by the newer gcc. It is in
fact in the libdap package and I have reported it upstream. (and the libdap
package cannot be rebuilt in devel).
(In reply to comment #1)
> Do you know whats the matter with devel at the moment? I realize its
> not a blocker, I'm just curious whats causing the problem -- and don't
> have a lot of time right now to look at the syntax...
for an explanation of the problem, as Patrice tells this is a problem of the
code and the fix is simple, remove the offending code. :-)
This is fixed in upstream libdap svn. It also seems that a new libdap version is
on tracks. I'll update libdap as soon as it is out and this should build in
Ok, glad to hear the gcc problem is being fixed upstream. Please change
the license to LGPL before checking it in and, since I don't see any other
blockers (comment #1), this package is APPROVED.