[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #13 from Fedora Update System  ---
python-fiona-1.7.8-2.fc27 has been pushed to the Fedora 27 stable repository.
If problems still persist, please make note of it in this bug report.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2017-09-28 15:57:22



--- Comment #12 from Fedora Update System  ---
python-fiona-1.7.8-2.fc26 has been pushed to the Fedora 26 stable repository.
If problems still persist, please make note of it in this bug report.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #11 from Fedora Update System  ---
python-fiona-1.7.8-2.fc26 has been pushed to the Fedora 26 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2017-0180c5666a

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
python-fiona-1.7.8-2.fc27 has been pushed to the Fedora 27 testing repository.
If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2017-89f565ab8d

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #9 from Fedora Update System  ---
python-fiona-1.7.8-2.fc27 has been submitted as an update to Fedora 27.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-89f565ab8d

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #8 from Fedora Update System  ---
python-fiona-1.7.8-2.fc26 has been submitted as an update to Fedora 26.
https://bodhi.fedoraproject.org/updates/FEDORA-2017-0180c5666a

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-19 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #7 from Ralph Bean  ---
(fedrepo-req-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/python-fiona

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #6 from Elliott Sales de Andrade  ---
Still need SCM. No-one has responded: https://pagure.io/releng/issue/6976

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-09-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #5 from Zbigniew Jędrzejewski-Szmek  ---
Can you build this? It's the last thing blocking python-geopandas.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-08-31 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877
Bug 1468877 depends on bug 1468873, which changed state.

Bug 1468873 Summary: Review Request: python-click-plugins - Click extension to 
enable registering CLI commands via setuptools entry-points
https://bugzilla.redhat.com/show_bug.cgi?id=1468873

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA



-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-08-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #4 from Zbigniew Jędrzejewski-Szmek  ---
Yes. It's also a bad description for pypi. (It contains too much
"advertisting", and doesn't really give the one to two sentence summary of what
this package "does".)

>>> print('\n'.join(sorted(fiona.drivers().drivers(
ARCGEN
AVCBin
AVCE00
AeronavFAA
AmigoCloud
BNA
CSV
CSW
Carto
Cloudant
CouchDB
DGN
DXF
EDIGEO
ESRI Shapefile
ElasticSearch
GFT
GML
GPKG
GPSBabel
GPSTrackMaker
GPX
GeoJSON
GeoRSS
Geoconcept
Geomedia
HTF
HTTP
Idrisi
Interlis 1
Interlis 2
JML
JP2OpenJPEG
JPEG2000
KML
LIBKML
MSSQLSpatial
MapInfo File
Memory
MySQL
NAS
ODBC
ODS
OGR_DODS
OGR_GMT
OGR_OGDI
OGR_PDS
OGR_SDTS
OGR_VRT
OSM
OpenAir
OpenFileGDB
PCIDSK
PDF
PGDUMP
PGeo
PLSCENES
PostgreSQL
REC
S57
SEGUKOOA
SEGY
SQLite
SUA
SVG
SXF
Selafin
TIGER
UK .NTF
VDV
VFK
WAsP
WFS
Walk
XLS
XLSX
XPlane
netCDF

That's way too many to list :(

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-08-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #3 from Elliott Sales de Andrade  ---
That's the first paragraph from PyPI: https://pypi.python.org/pypi/Fiona
Unfortunately, I cannot find a straightforward list of all supported formats.

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-08-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877

Zbigniew Jędrzejewski-Szmek  changed:

   What|Removed |Added

 Status|NEW |POST
 CC||zbys...@in.waw.pl
 Blocks|177841 (FE-NEEDSPONSOR) |
   Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl
  Flags||fedora-review+



--- Comment #2 from Zbigniew Jędrzejewski-Szmek  ---
It's generally better to not repeat the package name in Summary, since most of
the time the package name is shown separately anyway (by dnf list, etc.).

The %description is also a bit off in my opinion: the most important part is
what formats it can read and write. I'd move the last sentence to the beginning
and add some information about what formats it supports (I don't know how many
there are, but if it's possible, I'd just list them. This has the advantage
that it'll show up in searches.)

I'd also recommend the trick from
https://bugzilla.redhat.com/show_bug.cgi?id=1468873#c2 to avoid repeating the
lengthy description text.

But those are all non-blocking issues.

rpmlint:
python3-fiona.x86_64: W: no-manual-page-for-binary fio
That's not mandatory. Otherwise rpmlint says nothing useful.

+ package name is OK
+ license is acceptable for Fedora (BSD)
+ license is specified correctly
+ builds and install OK
+ %check is present and passes
+ modern python packaging template is used
+ R/P/BR look correct
+ %python_provide macros are present
+ %license macro is used
+ /usr/bin/fio runs OK

Package is APPROVED.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-07-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Blocks||1468995




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1468995
[Bug 1468995] Review Request: python-geopandas - Geographic pandas
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
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-07-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877



--- Comment #1 from Elliott Sales de Andrade  ---
Some informal reviews I have done:

https://bugzilla.redhat.com/show_bug.cgi?id=1440992#c1
https://bugzilla.redhat.com/show_bug.cgi?id=1440971#c1
https://bugzilla.redhat.com/show_bug.cgi?id=1433757#c2
https://bugzilla.redhat.com/show_bug.cgi?id=1368855#c14

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org


[Bug 1468877] Review Request: python-fiona - read and write spatial data files

2017-07-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1468877

Elliott Sales de Andrade  changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)
 Depends On||1468873




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
https://bugzilla.redhat.com/show_bug.cgi?id=1468873
[Bug 1468873] Review Request: python-click-plugins - Click extension to
enable registering CLI commands via setuptools entry-points
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org