[Bug 227669] Review Request: ppl - A modern C++ library providing numerical abstractions

[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: ppl - A modern C++ library providing numerical abstractions


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





------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx  2007-06-09 13:25 EST -------
Well, for 0.9-7:

* %check
  - This package contains test/ directory and some
    check programs seem to be executed. If "make check"
    makes sense, please add:
--------------------------------------------------
%check
make check
--------------------------------------------------
    (Well, this is very very long...)

* Documentation
  - It seems that README.configure is for people who want to
    rebuild ppl by themselves (i.e. who want to call configure
    by themselves) and so IMO this document should be removed
    for rpm user.

  - IMO TODO should be added to documents.

* %description
  - Please refrect static archive split. For example,
    -devel package says:
---------------------------------------------------
%description devel
The header files, documentation and static libraries for developing
                                ^^^^^^^^^^^^^^^^^^^^
---------------------------------------------------

* rpath issue
  - As said above, please let me know if rpath issue is
    resolved by hacking libtool.

All other things are okay.

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