Package Details: python-defcon 0.10.1-1

Git Clone URL: https://aur.archlinux.org/python-defcon.git (read-only, click to copy)
Package Base: python-defcon
Description: A set of UFO based objects for use in font editing applications
Upstream URL: https://github.com/robotools/defcon
Licenses: MIT
Submitter: wiill
Maintainer: thrasibule (alerque)
Last Packager: alerque
Votes: 4
Popularity: 0.39
First Submitted: 2017-01-05 01:27 (UTC)
Last Updated: 2022-05-07 08:05 (UTC)

Pinned Comments

alerque commented on 2021-08-03 20:16 (UTC)

PSA: Like most of the PKGBUILDs that I (co-)maintain, I host prebuilt packages for this in my user repository and all its dependencies for those who wish to install it using pacman without messing around with building from the AUR. Issues or contributions are welcome either in comments below or via this GitHub repository.

Latest Comments

alerque commented on 2021-08-03 20:16 (UTC)

PSA: Like most of the PKGBUILDs that I (co-)maintain, I host prebuilt packages for this in my user repository and all its dependencies for those who wish to install it using pacman without messing around with building from the AUR. Issues or contributions are welcome either in comments below or via this GitHub repository.

alerque commented on 2021-05-05 07:47 (UTC)

Thanks for the alert @djmattyg007. Please see my extended comments in this issue report and the related upstream bug report. I've posted temporary fixes so this builds in a clean chroot now.

djmattyg007 commented on 2021-04-24 05:39 (UTC)

This package currently fails to build because python-setuptools-scm isn't listed as a make dependency:

https://archlinux.org/packages/community/any/python-setuptools-scm/

It may look like pip is the missing make dependency, but this isn't the case. It's only trying to use pip because the real dependency is missing.

djmattyg007 commented on 2021-04-24 05:38 (UTC)

it is against current AUR guidelines to bump release just because they need rebuilding due to library updates in official repos

Where did you read this? I think it's plainly obvious that this isn't the case given the sheer number of packages that DO get pkgrel bumps in the first 48 hours after a new version of python is released.

Sadly makepkg -fi or yay --rebuild is your friend here.

I don't use yay, nor do I use makepkg directly. I use aurutils, which helps me run an actual package repository for all the machines in my house, so that I only have to compile the software once. This relies on pkgrel bumps to trigger updates on all machines.

Marcel_K commented on 2021-03-14 00:08 (UTC)

Isn't python-pip a (make)depend? I couldn't build this package without.

alerque commented on 2021-01-07 19:10 (UTC)

@djmattyg007 As much as I disagree with the policy of treating AUR pkgrel's different than ones in [community] in this regard, it is against current AUR guidelines to bump release just because they need rebuilding due to library updates in official repos. Sadly makepkg -fi or yay --rebuild is your friend here.

djmattyg007 commented on 2021-01-03 02:56 (UTC)

Could you please bump the pkgrel to force a rebuild for python3.9?

alerque commented on 2020-05-31 15:39 (UTC)

I still don't understand what is actually going on, but the issues with --check here are similar to what I just ran into with python-ufo2ft — the check function is actually trying to build something or other and wants an old version of fonttools.

alerque commented on 2020-05-31 13:15 (UTC)

Wow there IS something funky with this build. I can build this in a clean CHROOT but cannot on my host system! I think the test suite must be doing something really wonky looking for packages and adding tests if and only if the host system has extra things available. That's crazy pants but it's happening.

alerque commented on 2020-05-31 13:13 (UTC)

@thrasibule: Thanks for the add. I just normalized a few things following Arch's Python guidelines on the wiki and checked the build in a clean CHROOT.

@yochananmarqos I added you to the -git package, you probably mistook it for this one.

alerque commented on 2020-05-31 13:12 (UTC)

@andykluger I can't replicate your results needing fontmath or anything else not currently listed. I built the current iteration in a clean chroot starting with just base-devel plus the specified dependencies and it built fine. I'm wondering if those tests you are seeing failing are actually for building something else in the dependency chain.

andykluger commented on 2020-05-31 06:08 (UTC)

Ah, but if I remove both ufolib packages, and install python-fontmath, then this installs just fine.

It seems to me python-fontmath is necessary, at least at build time, as long as the check remains.

andykluger commented on 2020-05-31 06:01 (UTC)

This still fails for me in two places:

test_validation.py::UFOReadWriteValidateTest::test_ufoLibReadValidate_defaults
test_validation.py::UFOReadWriteValidateTest::test_customClasses

If I install python-ufolib, then it only fails in one place:

test_validation.py::UFOReadWriteValidateTest::test_customClasses

If I then additionally install python-ufolib2, it fails once:

test_validation.py::UFOReadWriteValidateTest::test_ufoLibReadValidate_defaults

If I then remove python-ufolib but keep python-ufolib2 installed, it fails once:

test_validation.py::UFOReadWriteValidateTest::test_customClasses

yochananmarqos commented on 2020-05-31 03:04 (UTC)

@thrasibuke: Apologies for the mix-up then. Thanks for updating it.

thrasibule commented on 2020-05-31 02:27 (UTC)

@yochananmarqos It is your imagination, I've never added you as a co-maintainer. There is zero functional change from 0.7.0 to 0.7.2. Taking 2 days to bump a package for a minor rev that fixes nothing is not the end of the world. @caleb I've added you as a co-maintainer. Thanks for the dependencies, I've updated that. I still think there is value in testing the package, and making sure it doesn't download extra dependencies in doing that, so I've kept the check.

yochananmarqos commented on 2020-05-31 02:06 (UTC)

@thrasibule: Was it my imagination or did you add @caleb and I as Co-Maintainers and then just as quickly remove us? Are you planning on maintaining the package or not?

Plexcon commented on 2020-05-29 01:19 (UTC)

FAILED Lib/defcon/test/objects/test_validation.py::UFOReadWriteValidateTest::test_ufoLibReadValidate_defaults - Error making: python-defcon

alerque commented on 2020-05-28 22:07 (UTC)

@thrasibule I think I have the dependencies for 0.7.0 right on the python-defcon-git package now. Note the docs and the actually source of setup.py don't quite match up. I still think that the check() probably just needs to be dropped altogether because it isn't really testing the build on Arch Linux, it is testing itself in an environment that is or can be different.

I'd still be willing to help maintain this and other font tooling related packages if you add me as a co-maintainer.

alerque commented on 2020-05-27 20:48 (UTC)

@yochananmarqos It's actually a bit more complicated than that because the listed dependencies make some assumptions about what those dependencies provide. Arch packages on the other hand don't always like up to that. The fonttools package in particular does not provide all the features out of the box unless you supply some of the extra dependencies.

Tests in Python are a different issue. Given that the test system seems to be bootstrapping it's own version of dependencies, it really is only checking the upstream code not the packaging. Testing that doesn't test against the system packages is pointless in a check() function. If that's the case we should just eschew it entirely.

yochananmarqos commented on 2020-05-27 15:18 (UTC) (edited on 2020-05-27 15:21 (UTC) by yochananmarqos)

python-fontpens and python-lxml are optional dependencies.

The tests also require python-unicodedata2, python-fs & python-pytz. If they're not installed, they're downloaded from PyPI.

The tests passed with v0.7.0 without python-ufolib or python-ufolib2. The dependencies require:

FontTools >= 3.31.0, installed with the fonttools[ufo] extra, required to import fonttools.ufoLib module.

It's referring to the module built in to fonttools, so nothing else should be required.

hwaiting commented on 2020-05-17 18:30 (UTC)

Even with python-ufolib2 it fails the tests on my system:

"Lib/defcon/test/objects/test_validation.py:58: AssertionError"

With python-ufolib as well it does pass the tests.

raxod502 commented on 2020-04-23 19:50 (UTC)

This needs a dependency on python-ufolib2, else the tests fail.

thrasibule commented on 2020-03-04 16:02 (UTC)

@caleb Can you tell me what's the issue that needs to be fixed? Last commit I've did was to bump the pkgrel. And package builds fine for me here.

alerque commented on 2020-03-04 06:56 (UTC) (edited on 2020-03-04 09:16 (UTC) by alerque)

@thrasibule This is still not fixed correctly. I commented on this last August, so did you, and you adopted ostensibly to fix the problem but the syntax you used is invalid. I commented on this in December and even though you added a commit since then you still didn't fix the original issue. Please either fix the dependency & syntax, orphan the package, or add me as a co-maintainer so this can be fixed properly. Thanks.

alerque commented on 2019-12-27 05:51 (UTC)

The last commit here fixes the previously bogus fixed dependency version (==), but it did not bump the package release so people like myself that installed in in the interim are unable to update without special arguments to force a rebuild. Please bump the package release number to reflect the change in dependencies so systems can update normally.

thrasibule commented on 2019-08-23 15:56 (UTC)

Actually it does depend on unicodedata2-12. If you don't have it installed, it downloads it.

thrasibule commented on 2019-08-23 15:54 (UTC)

I thought it depended on 12. I made a mistake, that happens... This package is not out of date, please don't flag it as such.

greyltc commented on 2019-08-23 14:56 (UTC)

And why do you specify ==12.0.0 anyway? No pytests here fail with unicodedata2 v11.

greyltc commented on 2019-08-23 14:51 (UTC)

probably don't use python-unicodedata2==12.0.0 if it does not exist

thrasibule commented on 2019-08-23 14:20 (UTC)

The dependency is there already. I've asked to bump the version in the AUR, and made an orphan request, but no replies yet.

alerque commented on 2019-08-23 11:55 (UTC)

This needs to require python-unicodedata2.