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

Test branch for Android 7 #3321

Closed
wants to merge 11 commits into from
Closed

Test branch for Android 7 #3321

wants to merge 11 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 8, 2019

A branch with minimal API level 24 to test some things before dropping Android 5/6 support.

@ghost ghost added the WIP Work in progress, do not close the issue (PR) label Feb 8, 2019
@ghost ghost force-pushed the android-7-testing branch 4 times, most recently from fcd8772 to eb75f09 Compare February 9, 2019 20:35
@ghost
Copy link
Author

ghost commented Feb 15, 2019

Compiled all packages except:

  • gst-plugins-good - build error
  • imagemagick - build error
  • php - tells that iconv library should be reinstalled
  • rust - just skipped
  • zbar - depends on imagemagick which cannot be compiled

@fornwall
Copy link
Member

php - tells that iconv library should be reinstalled

This was a problem on the master branch and has just been fixed there.

@Grimler91
Copy link
Member

Grimler91 commented Feb 16, 2019

@xeffyr what error(s) did you get for imagemagick? Something like

termux-build/imagemagick/src/MagickCore/fourier.c:815:34: error: passing 'fftw_complex' (aka 'double [2]') to parameter of incompatible type '_Complex double'
        magnitude_pixels[i]=cabs(forward_pixels[i]);
                                 ^~~~~~~~~~~~~~~~~

?

@ghost
Copy link
Author

ghost commented Feb 16, 2019

Yes, I get this error.

@Grimler91
Copy link
Member

Grimler91 commented Feb 16, 2019

@xeffyr Alright, should be fixed with #3350.

Is it okay if I force-push a rebased branch (https://github.com/Grimler91/termux-packages/tree/android-7-testing-rebased) or will that mess up bintray/gitlab-CI?

@Grimler91
Copy link
Member

@xeffyr Which error did you get for gst-plugins-good? It works for me on rebased branch for all arches

@ghost
Copy link
Author

ghost commented Feb 16, 2019

@Grimler91 It's

  CCLD     libgstpng.la
gstvpxdec.c:65:15: error: use of undeclared identifier 'VP8_DEBUG_TXT_FRAME_INFO'
    {C_FLAGS (VP8_DEBUG_TXT_FRAME_INFO),
              ^
gstvpxdec.c:68:15: error: use of undeclared identifier 'VP8_DEBUG_TXT_MBLK_MODES'
    {C_FLAGS (VP8_DEBUG_TXT_MBLK_MODES),
              ^
gstvpxdec.c:71:15: error: use of undeclared identifier 'VP8_DEBUG_TXT_DC_DIFF'
    {C_FLAGS (VP8_DEBUG_TXT_DC_DIFF),
              ^
gstvpxdec.c:74:15: error: use of undeclared identifier 'VP8_DEBUG_TXT_RATE_INFO'
    {C_FLAGS (VP8_DEBUG_TXT_RATE_INFO),
              ^
4 errors generated.

@ghost
Copy link
Author

ghost commented Feb 16, 2019

or will that mess up bintray/gitlab-CI?

Only force-push to protected branches will mess gitlab ci. I.e. master branch.

@Grimler91 Grimler91 force-pushed the android-7-testing branch 2 times, most recently from 5002fe1 to ca82ad3 Compare February 16, 2019 15:58
@ghost ghost mentioned this pull request Feb 16, 2019
@ghost ghost force-pushed the android-7-testing branch 2 times, most recently from ac5e00f to 0dcda5b Compare February 17, 2019 16:26
Grimler91 and others added 10 commits February 17, 2019 18:32
DT_RUNPATH will help mpv to find correct libraries.
Unsetting of `LD_LIBRARY_PATH` still needed.
To fix problem with uploading package to Bintray. Previous one
stucked with impossibility to upload i686 package and API failure.
@ghost
Copy link
Author

ghost commented Feb 19, 2019

Now everything from packages/ (except pkgs with custom license) is available for installation from https://dl.bintray.com/xeffyr/android-7-packages.

@ghost
Copy link
Author

ghost commented Feb 24, 2019

Further work will be done in https://github.com/termux/termux-packages/tree/android-7.

@ghost ghost closed this Feb 24, 2019
@ghost ghost removed the WIP Work in progress, do not close the issue (PR) label Feb 24, 2019
@ghost ghost deleted the android-7-testing branch February 24, 2019 23:02
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants