Fix cmmnbuild-dep-manager stub generation, and add appropriate tests.
-
Adds a new
_site_packagesmethod on Manager (it could have been moved out of Manager, but I kept it there for consistency with_dir,_user_dirand_dist_dir. -
Makes stubgen an optional package
pip install cmmnbuild-dep-manager[stubgen] -
I looked at enabling subgen by default if installed, but was concerned with what the implications would be given that it starts the JVM (prevents
Manager('pyjapc')typeresolvecalls from having control of JVM startup). To be investigated further. -
Fixes the breakage as a result of merging the move from
Manager._registered_packagestoManager._registered_modules. -
Adds CI testing for the stubgen integration (JPype 1.2 only)
Merge request reports
Activity
added 1 commit
- ba653d73 - Fix cmmnbuild-dep-manager stub generation, and add appropriate tests.
The changes look good to me, in tests there seems to be a little problem (related to some path mangling?)
Yeah, trying to track that down now. What do you mean by path mangling? I can only assume that there is some weird file permissions going on somewhere, but happy to take ideas if you have some!
Resolved by Philip ElsonFrom the GitLab test report, I first thought that one path had
__w0in the end while the other had not. But looking at the full log output, this is not the case...As what user are we running in the CI docker container? Non-root I hope?
- Last reply by Philip Elson