[Bug 1173683] Review Request: postgresql-semver - A semantic version data type for PostgreSQL

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

 



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

Pavel Raiskup <praiskup@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |praiskup@xxxxxxxxxx
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |praiskup@xxxxxxxxxx



--- Comment #1 from Pavel Raiskup <praiskup@xxxxxxxxxx> ---
Thanks for packaging.  I'll do the review soon, but for now, could we keep the
upstream package name, pg-semver in particular?  We tend not to create yet
another packages which looks like it comes from 'postgresql.spec'.  And it
is also less confusing for new users.

Also, could you please use module-guard require like (rawhide):
postgresql-server(:MODULE_COMPAT_%{postgresql_major})

The %defattr should not be needed nowadays.

Pavel

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





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