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

build(bindings/java): prepare for automatically multiple platform deploy #2335

Merged
merged 32 commits into from
May 27, 2023
Merged

build(bindings/java): prepare for automatically multiple platform deploy #2335

merged 32 commits into from
May 27, 2023

Conversation

tisonkun
Copy link
Member

This closes #2313.

Although, formal release still needs development.

tisonkun added 24 commits May 26, 2023 00:20
@tisonkun
stick to JDK 8
39b2ab9 
Signed-off-by: tison <[email protected]>
@tisonkun
tmp disable other workflows
c7ca7f4 
Signed-off-by: tison <[email protected]>
@tisonkun
try auto deploy
98c0097 
Signed-off-by: tison <[email protected]>
@tisonkun
tmp add this branch
8624033 
Signed-off-by: tison <[email protected]>
@tisonkun
try add token
386969f 
Signed-off-by: tison <[email protected]>
@tisonkun
simplify properties resolve logic
9f5dca2 
Signed-off-by: tison <[email protected]>
@tisonkun
build with script
b9522f8 
Signed-off-by: tison <[email protected]>
@tisonkun
correct script
0e2e73a 
Signed-off-by: tison <[email protected]>
@tisonkun
support cross compile for darwin
fedec64 
Signed-off-by: tison <[email protected]>
@tisonkun
deploy profile
e65bda0 
Signed-off-by: tison <[email protected]>
@tisonkun
fixup
a4441ba 
Signed-off-by: tison <[email protected]>
@tisonkun
add support for more platforms
8ba67a1 
Signed-off-by: tison <[email protected]>
@tisonkun
fix typo
9c9655d 
Signed-off-by: tison <[email protected]>
@tisonkun
workaround pwsh issue
e60aa7c 
Signed-off-by: tison <[email protected]>
@tisonkun
add py dep
b6b3cd4 
Signed-off-by: tison <[email protected]>
@tisonkun
workaround windows
088d129 
Signed-off-by: tison <[email protected]>
@tisonkun
skip tests
ea4970c 
Signed-off-by: tison <[email protected]>
@tisonkun
improve docs
b0313fb 
Signed-off-by: tison <[email protected]>
@tisonkun
try with github package
1a6b1f3 
Signed-off-by: tison <[email protected]>
@tisonkun
add github repo info
73253ef 
Signed-off-by: tison <[email protected]>
@tisonkun
update meta
0a5857e 
Signed-off-by: tison <[email protected]>
@tisonkun
Revert "tmp disable other workflows"
e4ba7bd 
This reverts commit c7ca7f4.
@tisonkun
revert trigger
d11e061 
Signed-off-by: tison <[email protected]>
@tisonkun
add docs
c42114a 
Signed-off-by: tison <[email protected]>
@tisonkun
fix typo
a44fee0 
Signed-off-by: tison <[email protected]>
@tisonkun tisonkun requested a review from suyanhanx May 26, 2023 13:40
@tisonkun
fix classifier
a9c1fec 
Signed-off-by: tison <[email protected]>
suyanhanx
suyanhanx previously approved these changes May 26, 2023
View reviewed changes
Copy link
Member

@suyanhanx suyanhanx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@tisonkun
Copy link
Member Author

GitHub has some troubles to work with classfier with dash - https://github.com/orgs/community/discussions/49682

We are in the case since the classfier is osx-aarch_64 or so. Damn it >_<

@tisonkun tisonkun marked this pull request as draft May 26, 2023 14:37
@tisonkun
Copy link
Member Author

Convert to draft and wait a bit for GitHub responses.

@Xuanwo Xuanwo dismissed suyanhanx’s stale review May 26, 2023 15:27

PR is still draft.

@tisonkun
Copy link
Member Author

@Xuanwo @suyanhanx Updated and revert the actual deploy workflow. I'll investigate other approach like:

  1. Use a third-party CI repo for automation
  2. Follow the suggestion on [email protected] to set up a Jenkins workflow.

GitHub Packages has one more shortcoming that we cannot depend on it https://github.com/orgs/community/discussions/48971.

@tisonkun tisonkun marked this pull request as ready for review May 27, 2023 03:33
@tisonkun tisonkun changed the title ci(bindings/java): automatically deploy snapshot build(bindings/java): support multiple platform build May 27, 2023
@tisonkun tisonkun changed the title build(bindings/java): support multiple platform build build(bindings/java): prepare for automatically multiple platform build May 27, 2023
@tisonkun tisonkun changed the title build(bindings/java): prepare for automatically multiple platform build build(bindings/java): prepare for automatically multiple platform deploy May 27, 2023
@tisonkun
drop more GH things
b19cb26 
Signed-off-by: tison <[email protected]>
@tisonkun
Copy link
Member Author

Current deploy snapshot in a 3rd party automation - https://github.com/tisonkun/ci-opendal/actions/runs/5096859042

@tisonkun tisonkun requested review from Xuanwo and suyanhanx May 27, 2023 04:17
Xuanwo
Xuanwo approved these changes May 27, 2023
View reviewed changes
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@Xuanwo Xuanwo merged commit c1305cc into apache:main May 27, 2023
@tisonkun tisonkun deleted the try-github-packages branch May 27, 2023 07:11
@Xuanwo Xuanwo mentioned this pull request May 30, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Xuanwo Xuanwo Xuanwo approved these changes

@PsiACE PsiACE Awaiting requested review from PsiACE

@ClSlaid ClSlaid Awaiting requested review from ClSlaid

@sundy-li sundy-li Awaiting requested review from sundy-li

@suyanhanx suyanhanx Awaiting requested review from suyanhanx

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Deploy Java bindings for multiple platforms
3 participants
@tisonkun @Xuanwo @suyanhanx