[Bug 1403314] Review Request: python-publicsuffix - Get a public suffix for a domain name using the Public Suffix List.

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

 



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

Merlin Mathesius <mmathesi@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |needinfo?(sgallagh@redhat.c
                   |                            |om)



--- Comment #3 from Merlin Mathesius <mmathesi@xxxxxxxxxx> ---
Thank you for addressing my questions and concerns.

I verified the updated package functions as described.

I now have a few additional questions/concerns:

* Is the intent that
/usr/lib/python*/site-packages/publicsuffix/public_suffix_list.dat be a symlink
to /usr/share/publicsuffix/public_suffix_list.dat, or a regular file copy of
public_suffix_list.dat picked up from the publicsuffix-list package at RPM
build time? It was a link in the previous version of this package, now it's a
regular file.

* The %changelog version-revision tag is now 1.1.0-1, but based on the Version
and Release, it should be 1.1.0-0.7. I presume this is anticipation of bumping
the Release to 1 upon actual release?

* I just noticed the warning that rpmlint doesn't like the Summary: to end with
a period. It's just a warning.

-- 
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
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




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