New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move qubes.qdb module outside qubes package #1252

Closed
woju opened this Issue Sep 29, 2015 · 3 comments

Comments

Projects
None yet
3 participants
@woju
Member

woju commented Sep 29, 2015

qubes.qdb, a compiled module, should still reside in /usr/lib64. It however
prevents importing of main qubes package, since /usr/lib64 is searched
before /usr/lib. Therefore it should be renamed, for example to qubesdb.

@woju woju added this to the Release 4.0 milestone Sep 29, 2015

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Sep 29, 2015

Member

Some time ago you've pointed some module to automatically generate
C->python bindings. Can you remind me what it was?
It may be also converted to use ctypes to access that library, but I'm
not sure if that would be better (but it will be easier to read).
Not saying that strictly want to do that now.

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Member

marmarek commented Sep 29, 2015

Some time ago you've pointed some module to automatically generate
C->python bindings. Can you remind me what it was?
It may be also converted to use ctypes to access that library, but I'm
not sure if that would be better (but it will be easier to read).
Not saying that strictly want to do that now.

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

@woju

This comment has been minimized.

Show comment
Hide comment
@woju

woju Sep 29, 2015

Member

On Tue, Sep 29, 2015 at 03:58:13AM -0700, Marek Marczykowski-Górecki wrote:

Some time ago you've pointed some module to automatically generate
C->python bindings. Can you remind me what it was?
It may be also converted to use ctypes to access that library, but I'm
not sure if that would be better (but it will be easier to read).
Not saying that strictly want to do that now.

Cython. I already have some preliminary work on
that.

And no, ctypes is not easier to read. ctypes is, by some accounts,
less readable than plain C, plus it is easy to segfault the interpreter.
Cython with only minor attention make that very hard.

Member

woju commented Sep 29, 2015

On Tue, Sep 29, 2015 at 03:58:13AM -0700, Marek Marczykowski-Górecki wrote:

Some time ago you've pointed some module to automatically generate
C->python bindings. Can you remind me what it was?
It may be also converted to use ctypes to access that library, but I'm
not sure if that would be better (but it will be easier to read).
Not saying that strictly want to do that now.

Cython. I already have some preliminary work on
that.

And no, ctypes is not easier to read. ctypes is, by some accounts,
less readable than plain C, plus it is easy to segfault the interpreter.
Cython with only minor attention make that very hard.

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Sep 29, 2015

Member

On Tue, Sep 29, 2015 at 11:08:21AM -0700, Wojtek Porczyk wrote:

And no, ctypes is not easier to read. ctypes is, by some accounts,
less readable than plain C, plus it is easy to segfault the interpreter.
Cython with only minor attention make that very hard.

You just need to define argument types (aka rewrite header file into
python), for example:
https://bitbucket.org/mhallin/py-scrypt/src/119842611360837902b12b7c5aadef5387318f70/scrypt.py?at=default

Anyway I think you know better what can be used here.

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Member

marmarek commented Sep 29, 2015

On Tue, Sep 29, 2015 at 11:08:21AM -0700, Wojtek Porczyk wrote:

And no, ctypes is not easier to read. ctypes is, by some accounts,
less readable than plain C, plus it is easy to segfault the interpreter.
Cython with only minor attention make that very hard.

You just need to define argument types (aka rewrite header file into
python), for example:
https://bitbucket.org/mhallin/py-scrypt/src/119842611360837902b12b7c5aadef5387318f70/scrypt.py?at=default

Anyway I think you know better what can be used here.

Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

marmarek added a commit to marmarek/qubes-core-qubesdb that referenced this issue Jan 15, 2016

woju added a commit to woju/qubes-core-admin that referenced this issue Mar 2, 2016

woju added a commit to woju/qubes-core-admin that referenced this issue Mar 3, 2016

woju added a commit to woju/qubes-core-admin that referenced this issue Mar 3, 2016

@andrewdavidwong andrewdavidwong added the task label Apr 6, 2016

@marmarek marmarek closed this May 10, 2016

woju added a commit to woju/qubes-core-qubesdb that referenced this issue Jun 13, 2016

woju added a commit to woju/qubes-core-qubesdb that referenced this issue Oct 5, 2016

marmarek added a commit to marmarek/qubes-core-qubesdb that referenced this issue Nov 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment