[Bug 1099267] Review Request: drupal7-addressfield - Address Field defines a new field type to store intl addresses

[bugzilla at redhat.com]

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



--- Comment #2 from Shawn Iwinski <shawn.iwinski at gmail.com> ---
(In reply to Parag AN(पराग) from comment #1)
> Suggestions:
> 1) Group tag is not necessary in Fedora now and can be removed from spec
> file. See https://fedoraproject.org/wiki/Packaging:Guidelines#Group_tag

Drupal 7 packaging guidelines template
(https://fedoraproject.org/wiki/Packaging:Drupal7#Module) has the Group tag. 
Per overall packaging guidelines
(https://fedoraproject.org/wiki/Packaging:Guidelines#Group_tag) it can stay to
be compatible with EPEL and all Drupal pkgs are packaged for EPEL.



Additional item:

3) "Requires: php(language) >= 5.1" not required and should not be present. 
You do need to run phpcompatinfo on the module though -- see
https://fedoraproject.org/wiki/Packaging:Drupal7#PHP_Extensions

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