Bug 214634

Summary:	textproc/py-sphinx: plist is incomplete
Product:	Ports & Packages	Reporter:	Raphael Kubo da Costa <rakuco>
Component:	Package Infrastructure	Assignee:	Kris Moore <kmoore>
Status:	Closed FIXED
Severity:	Affects Only Me	CC:	kmoore, python
Priority:	---	Flags:	rakuco: maintainer-feedback+
Version:	Latest
Hardware:	Any
OS:	Any

Description Raphael Kubo da Costa freebsd_committer

2016-11-18 19:10:07 UTC

textproc/py-sphinx fails `make check-plist`:

===> Checking for items in pkg-plist which are not in STAGEDIR
Error: Missing: %%PYTHON_SITELIBDIR%%/sphinx/pycode/Grammar-py2-sphinx1.4.pickle

USE_PYTHON=autoplist does not take into account that that .pickle file that is installed manually in post-install.

Comment 1 commit-hook freebsd_committer

2016-11-18 20:47:30 UTC

A commit references this bug:

Author: kmoore
Date: Fri Nov 18 20:46:43 UTC 2016
New revision: 426339
URL: https://svnweb.freebsd.org/changeset/ports/426339

Log:
   - Fix issue with pkg-plist and missing pickle

  PR: 214634
  Reported by: rakuco@

Changes:
  head/textproc/py-sphinx/Makefile

Comment 2 Kris Moore freebsd_committer

2016-11-18 20:49:00 UTC

Not sure how I missed that before, seemed to pass the test on my laptop anyway. Thanks for the heads up!

Comment 3 Raphael Kubo da Costa freebsd_committer

2016-11-18 21:38:26 UTC

Are you sure this is the right fix? That post-install was added in ports r301002 to avoid some bug. Plus, this commit will change the generated package so PORTREVISION should probably be bumped.

Comment 4 Kris Moore freebsd_committer

2016-11-19 02:08:50 UTC

I did test the new port, and it does indeed seem to install a .pickle file and builds our docs just fine. I'm wondering if its automatically building it now, which make the extra post-install step unnecessary (and resulted in the extra plist item)

My pkg info -l py27-sphinx shows the following file that indicated it did indeed build without the post-install:

/usr/local/lib/python2.7/site-packages/sphinx/pycode/Grammar-py2-sphinx1.4.pickle

As for PORTREVISION, I had thought the port was broken / not building packages due to plist error. However if that's not the case I'll be happy to bump it.