Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove BUILD_PY from build logic #430

Closed
5 of 7 tasks
devinrsmith opened this issue Apr 20, 2021 · 0 comments · Fixed by #747
Closed
5 of 7 tasks

Remove BUILD_PY from build logic #430

devinrsmith opened this issue Apr 20, 2021 · 0 comments · Fixed by #747
Assignees
@niloc132
Labels
build clean up
Milestone
June 2021

Comments

@devinrsmith
Copy link
Member

devinrsmith commented Apr 20, 2021 •
edited by niloc132
Loading

Relates to #429

  • /build.gradle uses this flag to decide if we can build an RPM - probably safe to delete
  • /gradle/deephaven-jpy.gradle uses this flag to delete maven generated output after the build - safe to delete along with everything else
  • /DB-test/build.gradle sets up a java test, with extra heap - this probably needs to be migrated to run the test in a docker container with the wheels installed
  • Generators/Generators.gradle generates intermediate JSON for python docs, and optionally depends on these tasks from other generate* tasks. Fixed in #530
  • Integrations/build.gradle sets up a java test, also needs to be migrated to run the test in a docker container with wheels installed. Fixed in #530
  • py/jpy-integration/build.gradle has many tasks, needs further investigation
  • sphinx/sphinx.gradle sphinx needs to be moved to docker in a container with wheels installed
@devinrsmith devinrsmith added this to the Backlog milestone Apr 20, 2021
@devinrsmith devinrsmith self-assigned this Apr 20, 2021
@niloc132 niloc132 assigned niloc132 and unassigned devinrsmith Apr 26, 2021
niloc132 added a commit to niloc132/deephaven-core that referenced this issue May 10, 2021
@niloc132
WIP deephaven#430, with a few hacks - commit needs to be rewritten
5d6f09f
niloc132 added a commit to niloc132/deephaven-core that referenced this issue May 10, 2021
@niloc132
WIP deephaven#430, with a few hacks - commit needs to be rewritten
babcf7a
@niloc132 niloc132 mentioned this issue May 10, 2021
Merged
niloc132 added a commit to niloc132/deephaven-core that referenced this issue May 10, 2021
@niloc132
WIP deephaven#430, with a few hacks - commit needs to be rewritten
78dd498
niloc132 added a commit to niloc132/deephaven-core that referenced this issue May 10, 2021
@niloc132
WIP deephaven#430, with a few hacks - commit needs to be rewritten
f38fd10
@niloc132 niloc132 modified the milestones: Backlog, May 2021 May 11, 2021
@niloc132 niloc132 mentioned this issue May 12, 2021
Merged
niloc132 added a commit to niloc132/deephaven-core that referenced this issue May 14, 2021
@niloc132
JPY integration test should run in docker
d6c371a 
This fixes all non-junit tasks that are part of the jpy-integration
project, so that they run during a normal test build, including the
path/debug mains.

Partial deephaven#430
niloc132 added a commit that referenced this issue May 14, 2021
@niloc132
JPY integration test should run in docker (#591)
529ef0d 
This fixes all non-junit tasks that are part of the jpy-integration
project, so that they run during a normal test build, including the
path/debug mains.

Partial #430
niloc132 added a commit to niloc132/deephaven-core that referenced this issue May 18, 2021
@niloc132
Support for jpy integration junit tests
51137a9 
Partial deephaven#430
@niloc132 niloc132 mentioned this issue May 18, 2021
Merged
niloc132 added a commit to niloc132/deephaven-core that referenced this issue May 21, 2021
@niloc132
Support for jpy integration junit tests
8dec7ae 
Partial deephaven#430
niloc132 added a commit that referenced this issue May 21, 2021
@niloc132
Support for jpy integration junit tests (#632)
8b7afdf 
Partial #430
@pete-petey pete-petey modified the milestones: May 2021, June 2021 Jun 2, 2021
niloc132 added a commit to niloc132/deephaven-core that referenced this issue Jun 9, 2021
@niloc132
Run remaining tests in docker so python is available
662ca81 
Note that all tests are presently ignored, but they do start up
properly.

Partial deephaven#430
@niloc132 niloc132 mentioned this issue Jun 9, 2021
Closed
niloc132 added a commit to niloc132/deephaven-core that referenced this issue Jun 9, 2021
@niloc132
Run remaining tests in docker (so python is available)
1be3c45 
Note that all tests are presently ignored, but they do start up
properly.

Partial deephaven#430
@niloc132 niloc132 mentioned this issue Jun 9, 2021
Merged
niloc132 added a commit that referenced this issue Jun 14, 2021
@niloc132
Run remaining tests in docker (so python is available) (#735)
d790c61 
Note that all tests are presently ignored, but they do start up
properly.

Partial #430
niloc132 added a commit to niloc132/deephaven-core that referenced this issue Jun 14, 2021
@niloc132
Remove the rest of BUILD_PY and the old python build wiring
95e0c42 
Fixes deephaven#430
@niloc132 niloc132 mentioned this issue Jun 14, 2021
Merged
niloc132 added a commit to niloc132/deephaven-core that referenced this issue Jun 15, 2021
@niloc132
Remove the rest of BUILD_PY and the old python build wiring
a048fd5 
Fixes deephaven#430
@niloc132 niloc132 mentioned this issue Jun 21, 2021
Merged
niloc132 added a commit that referenced this issue Jun 22, 2021
@niloc132
Update python doc gen for java11 javadoc format (#768)
ede96a7 
Partial #430
niloc132 added a commit to niloc132/deephaven-core that referenced this issue Jun 22, 2021
@niloc132
Remove the rest of BUILD_PY and the old python build wiring
ec9507c 
Fixes deephaven#430
niloc132 added a commit that referenced this issue Jun 23, 2021
@niloc132
Remove the rest of BUILD_PY and the old python build wiring (#747)
83c6397 
Fixes #430
jmao-denver pushed a commit to jmao-denver/deephaven-core that referenced this issue Apr 11, 2022
@niloc132 @jmao-denver
Support for jpy integration junit tests (deephaven#632)
6dc09b9 
Partial deephaven#430
jmao-denver pushed a commit to jmao-denver/deephaven-core that referenced this issue Apr 11, 2022
@niloc132 @jmao-denver
Run remaining tests in docker (so python is available) (deephaven#735)
824fd4d 
Note that all tests are presently ignored, but they do start up
properly.

Partial deephaven#430
jmao-denver pushed a commit to jmao-denver/deephaven-core that referenced this issue Apr 11, 2022
@niloc132 @jmao-denver
Update python doc gen for java11 javadoc format (deephaven#768)
182041c 
Partial deephaven#430
jmao-denver pushed a commit to jmao-denver/deephaven-core that referenced this issue Apr 11, 2022
@niloc132 @jmao-denver
Remove the rest of BUILD_PY and the old python build wiring (deephave…
76e31ac 
…n#747)

Fixes deephaven#430
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@niloc132 niloc132

Labels
build clean up
Projects
None yet
Milestone
June 2021
Development

Successfully merging a pull request may close this issue.

Remove the rest of BUILD_PY and the old python build wiring niloc132/deephaven-core
3 participants
@niloc132 @devinrsmith @pete-petey