audio/pithos: update from 1.0.0 to 1.4.1
Details
- Reviewers
araujo mat miwi - Commits
- rP491500: audio/pithos: Update to 1.4.1
Tested with poudriere for 11.2 and 12.0
Diff Detail
- Repository
- rP FreeBSD ports repository
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 22142 Build 21362: arc lint + arc unit
Event Timeline
Remove both pkg-install and pkg-deinstall and use:
GLIB_SCHEMAS= io.github.Pithos.gschema.xml
(and remove the entry from the pkg-plist file)
Also, all occurrences of 36 in the plist are wrong, because we have Python 3.4, 3.5, 3.6 and 3.7 in the tree.
audio/pithos/Makefile | ||
---|---|---|
13–16 | Why +=? | |
18–20 | Why +=? | |
35–42 | Use ${REINPLACE_CMD} and not ${SED}. Also, are you aware that you can pass multiple arguments to sed so that it processes many files at once? |
audio/pithos/Makefile | ||
---|---|---|
13–16 | I apologize, this was just copied over from the prior version. I have fixed this. | |
18–20 | Same as above. | |
35–42 | Thank you. I was not aware of ${REINPLACE_CMD}. I have rearranged the sed to do multiple files at once. | |
35–42 | I was also not aware of the GLIB_SCHEMAS variable. Thanks for the tip. I have fixed everything you've mentioned. I am having an issue now since the pyc files are in the stage directory, but not in the pkg-plist. |
audio/pithos/Makefile | ||
---|---|---|
42 | Would it not easier to use SHEBANG? |
audio/pithos/Makefile | ||
---|---|---|
18–20 | You set already USES= gnome, I think USE_GNOME=glib20 will work here too. |
audio/pithos/Makefile | ||
---|---|---|
42 | Hi, Take a look here: https://www.freebsd.org/doc/en/books/porters-handbook/uses-shebangfix.html |
audio/pithos/Makefile | ||
---|---|---|
37–47 | sed can do more than one thing. ${REINPLACE_CMD} -i '' -e \ 's|@@PYTHONPREFIX_SITELIBDIR@@|${PYTHONPREFIX_SITELIBDIR}|g; \ s|@@PYTHON_PATH@@|${PYTHONBASE}/bin/python${PYTHON_VER}|g' \ ${WRKSRC}/meson.build \ ${WRKSRC}/bin/pithos.in \ ${WRKSRC}/meson_post_install.py \ ${WRKSRC}/docs/conf.py |
- Remove glib20 from dependencies in favor of USE_GNOME
- Remove patch files in favor of sed
audio/pithos/Makefile | ||
---|---|---|
17 | Sphinx is in fact not used. I'm unsure why it is in the build, other than perhaps future use. I have removed the need for it. |