-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature: distribution of the package is cumbersome #17
base: develop
Are you sure you want to change the base?
Conversation
Signed-off-by: Isaac Saito <[email protected]>
dbcef20
to
285e210
Compare
…essary structure for Catkin/Ament pkg Signed-off-by: Isaac Saito <[email protected]>
…g on external libs such as cibuildwheel) Signed-off-by: Isaac Saito <[email protected]>
58873f7
to
844186a
Compare
Signed-off-by: Isaac Saito <[email protected]>
844186a
to
e2ca933
Compare
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
2dcf9ad
to
1fa4a49
Compare
Signed-off-by: Isaac Saito <[email protected]>
Got installed, but not executable :/
|
Signed-off-by: Isaac Saito <[email protected]>
…th Docker Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
c464c0d
to
354a04f
Compare
Even though the latest is 0.1.2 defined in log
|
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
cf7d142
to
93f4d0a
Compare
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
1c8ecf6
to
a93cbd1
Compare
Just for now, (manually) setting
|
Exec runtime error. Maybe main issue: Log
Ignoring this error for now. That happened inside of
Unlikely, it's probably due to the executable being executed inside of a Docker container (which is NOT an intended usecase). Indeed, on a host this prints. Yay!
|
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
0074aef
to
dd0dbf9
Compare
Wait, Log
Managed to resolve by https://stackoverflow.com/questions/27784271/how-can-i-use-setuptools-to-generate-a-console-scripts-entry-point-which-calls |
Signed-off-by: Isaac Saito <[email protected]>
dd0dbf9
to
dcbb967
Compare
Signed-off-by: Isaac Saito <[email protected]>
dcbb967
to
5543b33
Compare
Signed-off-by: Isaac Saito <[email protected]>
5543b33
to
2c733cf
Compare
Signed-off-by: Isaac Saito <[email protected]>
2c733cf
to
9e36bc7
Compare
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
Signed-off-by: Isaac Saito <[email protected]>
0108d47
to
fc4db9b
Compare
Issue aimed to address
Review items
docker-vcs-tools
is kind of unintended. Also has dash in it (almost illegal in Python). Maybe rename withdocker_vcs
ordocker_vc_lib
.setup.py
, I still get 0.1.0 fromtestpypi
(log in Feature: distribution of the package is cumbersome #17 (comment))