You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "Xuanwo (via GitHub)" <gi...@apache.org> on 2023/03/22 02:13:42 UTC
[GitHub] [incubator-opendal] Xuanwo opened a new issue, #1728: bindings/python: Switch to python/rust mixed layout
Xuanwo opened a new issue, #1728:
URL: https://github.com/apache/incubator-opendal/issues/1728
For pure Rust project layout, the `.so` file name has to be `opendal` because that's how Python `import` works. We can switch to mixed Python/Rust layout to change it to whatever we want. (And usually we use `_opendal` and re-export public APIs in Python `opendal` module).
_Originally posted by @messense in https://github.com/apache/incubator-opendal/pull/1205#discussion_r1144158907_
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-opendal] tisonkun commented on issue #1728: bindings/python: Switch to python/rust mixed layout
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on issue #1728:
URL: https://github.com/apache/incubator-opendal/issues/1728#issuecomment-1478884259
IMO C bindings can reserve the "libopendal.so" name. Other language binding cdylib is used by the specific language only so that we don't care about the lib name.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-opendal] Xuanwo commented on issue #1728: bindings/python: Switch to python/rust mixed layout
Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo commented on issue #1728:
URL: https://github.com/apache/incubator-opendal/issues/1728#issuecomment-1478828796
By this switching, we can:
- allow rust `opendal-python` export `libopendal_python.so`
- Export public API directly at python side.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-opendal] messense commented on issue #1728: bindings/python: Switch to python/rust mixed layout
Posted by "messense (via GitHub)" <gi...@apache.org>.
messense commented on issue #1728:
URL: https://github.com/apache/incubator-opendal/issues/1728#issuecomment-1478830316
I'll do it later.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-opendal] Xuanwo closed issue #1728: bindings/python: Switch to python/rust mixed layout
Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo closed issue #1728: bindings/python: Switch to python/rust mixed layout
URL: https://github.com/apache/incubator-opendal/issues/1728
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@opendal.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org