Skip to content
This repository has been archived by the owner on Dec 3, 2024. It is now read-only.

Upgrade gradle play publisher to 2.2.1 #1346

Merged
merged 3 commits into from
Jun 13, 2019
Merged

Conversation

imsodin
Copy link
Member

@imsodin imsodin commented Jun 13, 2019

Release deployment is broken, most likely as gradle play publisher 1.x is not compatible with gradle 5.x. Upgrading to 2.x requires running ./gradlew bootstrap due to extensive changes to how play metadata is handled and that requires having the key. @calmh Is it possible for you to run that command on teamcity or manually and commit the results to this branch?

@calmh
Copy link
Member

calmh commented Jun 13, 2019

I probably can, but I'm not sure where. The build runs in a fairly short lived docker container, and the agent container itself has no gradle stuff. Is this maybe something that should go in the docker/prebuild.sh or somewhere instead? I have zero clue how the gradle stuff works.

@imsodin
Copy link
Member Author

imsodin commented Jun 13, 2019

If you have "manual" access with the keys in place, then literally just checkout this branch and run ./gradlew bootstrap - no docker container required.

Otherwise maybe you can duplicate the "Release Syncthing-Android" build job and modify the custom script in step one to run ./gradlew --no-daemon bootstrap instead of ./gradlew --no-daemon buildNative lint .... No idea how you then get access to the generated files in app/src/main/play/..., maybe export the used docker image as an artifact and extract it from there.

@AudriusButkevicius
Copy link
Member

What does bootstrap do? Is it some local changes or remote changes?

@calmh
Copy link
Member

calmh commented Jun 13, 2019

Yeah, exactly. :) If this is just that it needs to have the keys in order to be able to set something up in the repo I think Audrius has those and could do it on some computer with the relevant software installed... (Doing this in the build container is probably cumbersome.)

@imsodin
Copy link
Member Author

imsodin commented Jun 13, 2019

It does change the directory/file structure of all the listings (descriptions and such) locally, but apparently it pulls the information from google play instead of using what's in the repo.

@calmh
Copy link
Member

calmh commented Jun 13, 2019

Maybe I can build the docker image locally and run it there. I have no interest in trying to satisfy all the SDK/NDK/Java/Studio/whatever requirements on my work mac :)

@calmh
Copy link
Member

calmh commented Jun 13, 2019

Or if you just lack the signing keys and have everything else in place, we can solve that. I don't think those keys give access to Play or have anything to do with that, and I think you're already on that project?

@imsodin
Copy link
Member Author

imsodin commented Jun 13, 2019

Docker should work, just takes a moment as it needs to build various ndk/go things.

And it's just the signing keys (I hope, I'd lie if I'd say I am sure about anything regarding android o.O ).

@calmh
Copy link
Member

calmh commented Jun 13, 2019

I'm experimenting.

@imsodin
Copy link
Member Author

imsodin commented Jun 13, 2019

This upgrade is a really nice experience.

/sarcasm off
Thanks for all the help with this!

@imsodin imsodin merged commit a8be7cd into syncthing:master Jun 13, 2019
@imsodin imsodin deleted the upgradePPP branch June 13, 2019 14:51
@Catfriend1
Copy link
Contributor

@imsodin:
I'm trying to follow this PR since several hours but having no luck with the gradlew publishListingRelease command. It halts with a http-500 status code error and no error message points at the reason. This is happening in the lang "ar" after it tries to upload images. In the play/listings/ar folder there are only .txt files (like in the official repo) and no PNG's , no "graphics"folder".
I saw "uploadImages = false" was removed as it is no longer available in the play-publisher plugin.
Do you have any clue why I cannot upload?

 com.google.api.client.googleapis.json.GoogleJsonResponseException: 500 Internal Server Error
  {
    "code" : 500,
    "message" : null
  }

@imsodin
Copy link
Member Author

imsodin commented Jun 17, 2019

Lets see whether something comes out of the upstream bug report. A workaround if it keeps failing would be just moving the play listings graphics to another location and updating them manually in the console when they change.

@syncthing syncthing locked and limited conversation to collaborators Jun 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants