[issue23287] ctypes.util.find_library needlessly call crle on Solaris

[Martin Panter]

Martin Panter added the comment:

Do you have an opinion of the proposal Shawn?

Judging by John’s “[the constant] is what the system provides, though
users may customize as needed”, it sounds like the patch is more than a bug 
fix. Will it also mean find_library() will no longer search customized library 
directories? I’m not familiar with Solaris, but it seems this may not be 
intended or desired.

Why is this bug marked as “resource usage”?

--
nosy: +martin.panter
stage:  -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23287] ctypes.util.find_library needlessly call crle on Solaris

[Shawn]

Shawn added the comment:

Could we get someone to evaluate this please?

--
nosy: +swalker

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23287] ctypes.util.find_library needlessly call crle on Solaris

[John Beck]

John Beck added the comment:

First, there are two related but somewhat separate issues here.

Regarding the patches attached to http://bugs.python.org/issue20664
they seem fine.  In theory, they should not be needed, as though it
is true that dump(1) moved from /usr/ccs/bin to /usr/bin in Solaris
11, /usr/ccs/bin still exists as a sym-link to /usr/bin.  But the
patches are written in a cautious manner, so the Right Thing [tm]
should happen in all circumstances.

Regarding my assertion that 'the default library path is a constant
on Solaris: /lib/64:/usr/lib/64 in 64-bit mode and /lib:/usr/lib
in 32-bit mode', I stand by that but will clarify what I meant by
default.  What I meant was this is what the system provides, though
users may customize as needed.

I hope that helps.  If not, I'm happy to continue the conversation.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23287
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23287] ctypes.util.find_library needlessly call crle on Solaris

[Jeff Quast]

Jeff Quast added the comment:

John, What do you think of the patches attached to 
http://bugs.python.org/issue20664 ?

crle is not needed at all because the default library path is
a constant on Solaris

I don't believe this to be true, source? crle is absolutely needed to add 
additional library lookup paths on Solaris, did this recently change?

crle is most certainly especially in regards to zones: a zone is unable to 
modify any of the system library paths, it wouldn't be able to install any new 
libraries in those given paths (/usr/lib and /lib are often shared read-only by 
the global zone), and crle must be used to add a library path to a writable 
mountpoint, such as /usr/local/lib, and often /opt and other various deviations 
must occur to accommodate gnu tools, etc.

--
nosy: +jquast

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23287
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23287] ctypes.util.find_library needlessly call crle on Solaris

[Jeff Quast]

Jeff Quast added the comment:

I looked over the focus on default path, thank you for clarifying!

Sadly, I can't help you move either of these patches forward, best wishes!

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23287
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue23287] ctypes.util.find_library needlessly call crle on Solaris

[John Beck]

New submission from John Beck:

On Solaris, in Lib/ctypes/util.py, we have code that looks for
/usr/bin/crle and calls it to parse its output to try to determine
the Default Library Path.  This code broke recently (Solaris 12 build
65), as it expects to find a line starting with
Default Library Path (ELF):
but the  (ELF) part of that line was removed because it was no longer
needed.  So we need a change here regardless.  But it turns out that
calling crle is not needed at all because the default library path is
a constant on Solaris: /lib/64:/usr/lib/64 in 64-bit mode and
/lib:/usr/lib in 32-bit mode.  Thus I offer the attached patch
for both 2.7 and 3.4.

--
components: ctypes
files: crle-fix.patch
keywords: patch
messages: 234417
nosy: jbeck
priority: normal
severity: normal
status: open
title: ctypes.util.find_library needlessly call crle on Solaris
type: resource usage
versions: Python 2.7, Python 3.4
Added file: http://bugs.python.org/file37800/crle-fix.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23287
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com