[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: (I forgot that distutils2’s setup.py has its own fork of the conversion functions, for bootstrapping reasons, so I need to port the fix there.) -- ___ Python tracker ___

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: Thanks for the help Erik. -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker ___

[issue11805] package_data only allows one glob per-package

Roundup Robot added the comment: New changeset 83a0985c7aad by Éric Araujo in branch 'default': Allow multiple values for package_data in setup.cfg (#11805). http://hg.python.org/distutils2/rev/83a0985c7aad New changeset ea717d8e71d0 by Éric Araujo in branch 'python3': Merge fixes for #13901, #

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: > Note that my proposed syntax does not allow something equivalent to {'': > [etc.]} in > distutils, i.e. package data for top-level modules. I think this is okay: > modules should > not install data in their installation dir. I don’t think it was widely used

[issue11805] package_data only allows one glob per-package

Roundup Robot added the comment: New changeset 2c08bf9aca22 by Éric Araujo in branch 'default': Allow multiple values for package_data in setup.cfg (#11805). http://hg.python.org/cpython/rev/2c08bf9aca22 -- nosy: +python-dev ___ Python tracker

[issue11805] package_data only allows one glob per-package

Erik Bray added the comment: This patch works for me, and I'm happy with the syntax. Thanks! -- ___ Python tracker ___ ___ Python-bu

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: I figured it would let people comment on the syntax I propose more easily if I extracted it from the patch. Here’s the example: package_data = cheese = data/templates/* doc/* doc/images/*.png We have a package name, equals sign, then specs/globs

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: Note that my proposed syntax does not allow something equivalent to {'': [etc.]} in distutils, i.e. package data for top-level modules. I think this is okay: modules should not install data in their installation dir. I don’t think it was widely used, but maybe

[issue11805] package_data only allows one glob per-package

Changes by Éric Araujo : Added file: http://bugs.python.org/file24266/fix-package_data-multivalue-d2.diff ___ Python tracker ___ ___ Python-bu

[issue11805] package_data only allows one glob per-package

Changes by Éric Araujo : -- keywords: +patch Added file: http://bugs.python.org/file24265/fix-package_data-multivalue-cpy33.diff ___ Python tracker ___ _

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: Here are patches for CPython and the d2 repo. There is no doc update as the setupscript page still talks about setup.py and the setupcfg spec does not mention package_data at all (the negationists resources-promoters at work :), so this can wait for my big doc

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: (To clarify quotes in my message: First is a copied message from me, marked [I], then one form Erik, and the last paragraph is me again. Sleep good, should get more of that.) -- ___ Python tracker

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: Copied from #13712: [I] Problem is that the setup.cfg syntax does not define how to give more than one value. If it’s judged acceptable to disallow paths with embedded spaces, we could do something like this: [files] package_data = spam = first second thir

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: I need this to package test files for distutils2’s own next release, so pragmatism/compatibility will win for the short term :) I’ll fix it together with #13463 and #5302. -- priority: normal -> release blocker ___ Py

[issue11805] package_data only allows one glob per-package

Paul Moore added the comment: One important point - in the "new world" where data files living alongside code is unsupported, the bdist_msi and bdist_wininst installers need to be updated to install data files as needed. This may work already (I'll do some tests to see how well when I get back t

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: > Looking at sysconfig.cfg makes it all pretty clear, though it was hard to > find this > information a while ago. Yeah, you had to be here when the resources code was committed, or to stumble on it while reading the source code. It’s one of the big things I wil

[issue11805] package_data only allows one glob per-package

Paul Moore added the comment: I see what you're saying - and looking through sysconfig.cfg, I can see how things are expected to be laid out (and how I'd configure it if I didn't like it :-)) But as far as I can see, there's no way in packaging to describe a module that works like sysconfig

[issue11805] package_data only allows one glob per-package

Erik Bray added the comment: > When run from an uninstalled checkout, for example when developing, the > file will be found in the checkout. When run after being installed on > a Debian system, the file will be found in /usr/local/share > /spam/templates/log.txt. Each Python installation can

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: > As far as I've been able to tell there is no proposed syntax in the docs > specifically for > package_data. Right. I’ve only found an example in d2’s own setup.cfg: package_data = distutils2._backport = sysconfig.cfg distutils2.command = wininst*.exe

[issue11805] package_data only allows one glob per-package

Changes by Jeremy Kloth : -- nosy: +jkloth ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue11805] package_data only allows one glob per-package

Erik Bray added the comment: As far as I've been able to tell there is no proposed syntax in the docs specifically for package_data. The docs for the resources option seems to suggest separating globs with spaces, which would be fine by me (wouldn't allow paths that contain spaces, but that'

[issue11805] package_data only allows one glob per-package

Éric Araujo added the comment: What’s the syntax described in the docs? package_data = mypackage = templates/*.html static/css/*.css or package_data = mypackage = templates/*.html mypackage = static/css/*.css ? On a related subject, I think the new resources system sho

[issue11805] package_data only allows one glob per-package

New submission from Erik Bray : In distutils the package_data option can be supplied a list of glob patterns for each package. distutils2 currently only supports one glob per package. This could easily be fixed by simply allowing more than one `package_name = pattern` value in the package_dat