[Bug 452584] Review Request: mldonkey - Client for several P2P networks
bugzilla at redhat.com
bugzilla at redhat.com
Mon Jun 23 22:57:20 UTC 2008
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: mldonkey - Client for several P2P networks
https://bugzilla.redhat.com/show_bug.cgi?id=452584
------- Additional Comments From rpm at greysector.net 2008-06-23 18:57 EST -------
Partial review:
http://bugzilla.livna.org/show_bug.cgi?id=1487#c43 is still not addressed fully:
Since it doesn't run as root, it's better to move the log to
/var/log/mldonkey/mldonkey.log
and set ownership of that directory accordingly.
---
http://bugzilla.livna.org/show_bug.cgi?id=1487#c39 is still not addressed fully:
mldonkey_df_monitor.sh:
email=`grep "^ mail =" /var/lib/mldonkey/downloads.ini | sed 's/ mail = //g'`
This is far from foolproof. It'll fail there are whitespace changes in that line.
---
I see that the install section has improved a bit (uses make install now), but
it's still far from ideal. Though I guess there's nothing wrong with it
packaging-wise.
BuildRequires: dos2unix
Redundant BR, trivial to avoid using perl or tr. Not a blocker though.
It doesn't build on F-8, though:
error: Failed build dependencies:
ocaml-lablgtk-devel >= 2.10.0 is needed by mldonkey-2.9.5-1.fc8
If I relax that BR to >= 2.6.0, it fails at compilation:
ocamlopt.opt -inline 10 -I src/utils/cdk -I src/utils/lib -I
src/utils/ocamlrss -I src/utils/xml-light -I src/utils/net -I tools -I
src/daemon/common -I src/daemon/driver -I src/utils/mp3tagui -I
src/config/unix -I src/gtk2/gui -I src/gtk2/gui/x11 -I src/gtk2/gui/win32 -I
icons/rsvg -I +lablgtk2 -I src/networks/direct_connect -I
src/networks/fasttrack -I src/networks/fileTP -I src/networks/bittorrent -I
src/networks/donkey -c src/utils/net/ip_set.ml
/usr/bin/ocamlc.opt -I src/utils/cdk -I src/utils/lib -I src/utils/ocamlrss
-I src/utils/xml-light -I src/utils/net -I tools -I src/daemon/common -I
src/daemon/driver -I src/utils/mp3tagui -I src/config/unix -I src/gtk2/gui
-I src/gtk2/gui/x11 -I src/gtk2/gui/win32 -I icons/rsvg -I +lablgtk2 -I
src/networks/direct_connect -I src/networks/fasttrack -I src/networks/fileTP
-I src/networks/bittorrent -I src/networks/donkey -c src/utils/net/ip_set.ml
/usr/bin/ocamlc.opt -I src/utils/cdk -I src/utils/lib -I src/utils/ocamlrss
-I src/utils/xml-light -I src/utils/net -I tools -I src/daemon/common -I
src/daemon/driver -I src/utils/mp3tagui -I src/config/unix -I src/gtk2/gui
-I src/gtk2/gui/x11 -I src/gtk2/gui/win32 -I icons/rsvg -I +lablgtk2 -I
src/networks/direct_connect -I src/networks/fasttrack -I src/networks/fileTP
-I src/networks/bittorrent -I src/networks/donkey -c src/utils/net/http_server.mli
ocamlopt.opt -inline 10 -I src/utils/cdk -I src/utils/lib -I
src/utils/ocamlrss -I src/utils/xml-light -I src/utils/net -I tools -I
src/daemon/common -I src/daemon/driver -I src/utils/mp3tagui -I
src/config/unix -I src/gtk2/gui -I src/gtk2/gui/x11 -I src/gtk2/gui/win32 -I
icons/rsvg -I +lablgtk2 -I src/networks/direct_connect -I
src/networks/fasttrack -I src/networks/fileTP -I src/networks/bittorrent -I
src/networks/donkey -c src/utils/net/http_server.ml
ocamlopt.opt -inline 10 -I src/utils/cdk -I src/utils/lib -I
src/utils/ocamlrss -I src/utils/xml-light -I src/utils/net -I tools -I
src/daemon/common -I src/daemon/driver -I src/utils/mp3tagui -I
src/config/unix -I src/gtk2/gui -I src/gtk2/gui/x11 -I src/gtk2/gui/win32 -I
icons/rsvg -I +lablgtk2 -I src/networks/direct_connect -I
src/networks/fasttrack -I src/networks/fileTP -I src/networks/bittorrent -I
src/networks/donkey -c src/daemon/common/commonOptions.ml
Fatal error: out of memory.
make: *** [src/daemon/common/commonOptions.cmx] Error 2
error: Bad exit status from /home/rathann/build/tmp/rpm-tmp.20151 (%build)
Note that I have 2GB RAM, which I think should be enough. Or is that a problem
with lablgtk?
Builds fine in mock/devel/i386.
--
Configure bugmail: https://bugzilla.redhat.com/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