[Bug 702987] Review Request: stdair - C++ Standard Airline IT Library

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Martin Gieseking <martin.gieseking at uos.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|nobody at fedoraproject.org    |martin.gieseking at uos.de
               Flag|                            |fedora-review?

--- Comment #6 from Martin Gieseking <martin.gieseking at uos.de> 2011-07-30 15:50:47 EDT ---
(In reply to comment #5)
> As for the INSTALL file, the
> guidelines require to remove it:
> https://fedoraproject.org/wiki/Packaging/Guidelines#Documentation

To be nitpicking: The guidelines require not to package INSTALL files but you
don't need to remove it explicitly. If you don't add it as %doc, everything
should be fine. However, it doesn't hurt to remove it either. It's just
redundant. :)

> Again, many thanks for your first review elements, as that package is important
> for my team. It constitutes the ground work for twelve other components, which
> all put together offer a nice revenue management simulation engine
> (http://sourceforge.net/projects/dsim/).

OK, thanks for the information, and you're welcome. I'll do the formal review
if you're recent version is packaged.

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