[Bug 201551] Review Request: db4o

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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: db4o


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





------- Additional Comments From paul@xxxxxxxxxxxxxxxxxxxxxx  2006-08-08 03:07 EST -------
E: db4o no-binary
The package should be of the noarch architecture because it doesn't contain
any binaries.

I'll change that.

E: db4o only-non-binary-in-usr-lib
There are only non binary files in /usr/lib so they should be in /usr/share.

Mono packages aren't sane when it comes to this and until mono is fixed, this
one can be ignored

* documentation takes up 75% of the installed size, -docs subpackage is necessary

Not a problem - I did that with XaraLX

i.e. mono(System) and mono(mscorlib) are provided by mono-core
* documentation takes up 75% of the installed size, -docs subpackage is necessary

It needs mono-core as a R and mono-devel as a BR.

* package doesn't meet packaging guidelines
- {_prefix}/%{_lib} vs. %{_libdir}, See 

Permitted for the time being - have a look at the mono packaging guidelines

* Also, I don't like the use of sed where tr -d would suffice.

I'm not sure that's a blocker though

I'll fix the docs and noarch and upload a new spec file. Thanks for the review
up to now.

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

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]