all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#52834: sanity-check fails with namespace packages
@ 2021-12-27 20:49 Hartmut Goebel
  2021-12-28 14:03 ` Lars-Dominik Braun
  0 siblings, 1 reply; 2+ messages in thread
From: Hartmut Goebel @ 2021-12-27 20:49 UTC (permalink / raw)
  To: 52834; +Cc: Lars-Dominik Braun

Hi,

I just investigated some failing python packages 
<https://ci.guix.gnu.org/eval/16105/dashboard> and found that 
"python2-zppe-*" packages fail. (Most due to a dependency failing , 
though. Actually failing are python2-zope-testing and python2-zope-event).

These fail due to sanity-check not being able to import "zope" - which 
is a namespace package. Both use the "src directory layout" (source is 
contained in a sub-directory "src").

This could be solved by fetching a list og namespace-packages and 
checking whether a fails import is a namespace-package. Maybe there are 
other solution.

try:

      nspkgs = set(dist.get_metadata_lines('namespace_packages.txt'))

except:

     nspkgs = set()

Anyhow, since Python2 is EOL since long, I'm not sure whether it's worth 
the effort.

WDYT?

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |





^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#52834: sanity-check fails with namespace packages
  2021-12-27 20:49 bug#52834: sanity-check fails with namespace packages Hartmut Goebel
@ 2021-12-28 14:03 ` Lars-Dominik Braun
  0 siblings, 0 replies; 2+ messages in thread
From: Lars-Dominik Braun @ 2021-12-28 14:03 UTC (permalink / raw)
  To: Hartmut Goebel; +Cc: 52834

Hi Hartmut,

> These fail due to sanity-check not being able to import "zope" - which 
> is a namespace package. Both use the "src directory layout" (source is 
> contained in a sub-directory "src").
As far as I see PEP 420 (implicit namespace packages) is supported by
Python >=3.3 only, so I’m not sure the packages would work even if we
disabled 'sanity-check, do they? Either way, I’m in favor of removing
broken Python 2 packages.

> This could be solved by fetching a list og namespace-packages and 
> checking whether a fails import is a namespace-package. Maybe there are 
> other solution.
> […]
>       nspkgs = set(dist.get_metadata_lines('namespace_packages.txt'))
Depending on undocumented setuptools behavior should imo be avoided and
– for top_level.txt – phased out if possible.

Cheers,
Lars





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-12-28 14:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-27 20:49 bug#52834: sanity-check fails with namespace packages Hartmut Goebel
2021-12-28 14:03 ` Lars-Dominik Braun

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.