F-Droid build failed for version 0.54.0 #40

Closed
opened 12 months ago by marc.nause · 5 comments

According to https://gitlab.com/fdroid/fdroiddata/-/commit/4cae0c1cc72eed28fdc80d2bd2cbfed31d8d7991 F-Droid was not able to build version 0.54.0. The reason is:

cannot find symbol TEXT_PAINT

I checked https://codeberg.org/Starfish/TinyWeatherForecastGermany/src/branch/master/app/src/main/java/de/kaffeemitkoffein/tinyweatherforecastgermany/ForecastBitmap.java and it uses TEXT_PAINT, but it is not decared anywhere.

In my branch (https://codeberg.org/marc.nause/TinyWeatherForecastGermany/src/branch/fixjerkylist/app/src/main/java/de/kaffeemitkoffein/tinyweatherforecastgermany/ForecastBitmap.java) it is declared in line 39.

It seems like the manual merge in pull request #39 has not worked correctly.

According to [https://gitlab.com/fdroid/fdroiddata/-/commit/4cae0c1cc72eed28fdc80d2bd2cbfed31d8d7991](https://gitlab.com/fdroid/fdroiddata/-/commit/4cae0c1cc72eed28fdc80d2bd2cbfed31d8d7991) F-Droid was not able to build version 0.54.0. The reason is: >cannot find symbol TEXT_PAINT I checked https://codeberg.org/Starfish/TinyWeatherForecastGermany/src/branch/master/app/src/main/java/de/kaffeemitkoffein/tinyweatherforecastgermany/ForecastBitmap.java and it uses TEXT_PAINT, but it is not decared anywhere. In my branch (https://codeberg.org/marc.nause/TinyWeatherForecastGermany/src/branch/fixjerkylist/app/src/main/java/de/kaffeemitkoffein/tinyweatherforecastgermany/ForecastBitmap.java) it is declared in line 39. It seems like the manual merge in pull request https://codeberg.org/Starfish/TinyWeatherForecastGermany/pulls/39 has not worked correctly.
Owner

yes, just noticed yesterday night, too.

Strange thing. I had to manually merge your pull request, because I changed the ForecastBitmap.java the same time you did the pull request, which resulted in a conflict.

I used a patch-file, which I worked along step by step to merge both file versions and to join the code changes together.

I have a local copy that is working, but it does not match the one in this repo. I am not sure how this happened, because everything changed should have been pulled, and now I have to try to fix it somehow....

yes, just noticed yesterday night, too. Strange thing. I had to manually merge your pull request, because I changed the ForecastBitmap.java the same time you did the pull request, which resulted in a conflict. I used a patch-file, which I worked along step by step to merge both file versions and to join the code changes together. I have a local copy that is working, but it does not match the one in this repo. I am not sure how this happened, because everything changed should have been pulled, and now I have to try to fix it somehow....
Owner

ok, found out I had some (but not all) of the changes stashed, have committed them now, so it should be solved. 👍

I will fix 2 small bugs before a new release, since a next fdroid build try requires a new tag.

ok, found out I had some (but not all) of the changes stashed, have committed them now, so it should be solved. 👍 I will fix 2 small bugs before a new release, since a next fdroid build try requires a new tag.
Owner

Everything is fixed. I will test & release 0.54.1 soon.

Everything is fixed. I will test & release 0.54.1 soon.
Owner

New version tagged. Let's hope the best.

New version tagged. Let's hope the best.
Poster

I just successfully installed version 0.54.1 from F-Droid. 👍

I just successfully installed version 0.54.1 from F-Droid. 👍
marc.nause closed this issue 12 months ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.