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.

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

Shifts Kujaku to download from github packages #2475

Closed
wants to merge 2 commits into from

Conversation

shubham1g5
Copy link
Contributor

#2473
onaio/kujaku#352 (comment)

This will require devs to generate a personal access token from github and add github.properties

@shubham1g5 shubham1g5 added this to the 2.52 milestone May 12, 2021
Copy link
Contributor

@ShivamPokhriyal ShivamPokhriyal left a comment

Choose a reason for hiding this comment

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

Looks good

name = "GitHubPackages"
url = uri("https://maven.pkg.github.com/onaio/kujaku")
credentials {
username = githubProperties['gpr.usr'] ?: System.getenv("GPR_USER")
Copy link
Contributor

Choose a reason for hiding this comment

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

System.getenv("GPR_USER") is getting value from jenkins job config right?
If yes, Should we rather use gradle.properties here, since creating new jobs in jenkins would require adding this property.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good point, will update.

@ctsims
Copy link
Member

ctsims commented May 12, 2021

Oof, this is a really rough change. This adds a new dependency where anyone downloading the codebase needs a github packages PAT? Have we asked the Kujaku project maintainers to change that? I think this calls the dependency into question.

@shubham1g5
Copy link
Contributor Author

@ctsims that's true unfortunately, though I think Github Packages is a temporary workaround from them until they are able to migrate to maven central. @ekigamba Wondering if you will be open to shift to more open artifact managers like Jitpack for example as I assume migrating to maven central will take some time for you.

@ekigamba
Copy link

@ctsims @shubham1g5 Github packages is a temporary solution for now. We are open to any other artifact manager that is easy to integrate. We are open to any suggestions on the issue onaio/kujaku#352

@shubham1g5
Copy link
Contributor Author

Closing in favour of #2481

@shubham1g5 shubham1g5 closed this May 20, 2021
@shubham1g5 shubham1g5 deleted the kujakuDownloadFix branch May 20, 2021 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants