Skip to content

android-studio: update to 2022.3.1.18, orphan#45676

Merged
classabbyamp merged 1 commit into
void-linux:masterfrom
Luciogi:android-studio
Sep 27, 2023
Merged

android-studio: update to 2022.3.1.18, orphan#45676
classabbyamp merged 1 commit into
void-linux:masterfrom
Luciogi:android-studio

Conversation

@Luciogi

@Luciogi Luciogi commented Aug 19, 2023

Copy link
Copy Markdown
Contributor

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, (x86_64-glibc)

This PR is based on #41215

@Luciogi

Luciogi commented Aug 19, 2023

Copy link
Copy Markdown
Contributor Author

I have doubts:


For maintainer ship , I m not fully ready

Comment thread srcpkgs/android-studio/template Outdated
@Bnyro

Bnyro commented Aug 20, 2023

Copy link
Copy Markdown
Contributor

As described here, please add , orphan to your commit message.

@Bnyro

Bnyro commented Aug 20, 2023

Copy link
Copy Markdown
Contributor

This package has optional dependency on ncurses5, though application works without it

Optional dependencies should likely only be included if they're required for some fundamental features of the package - and I don't think that's the case here.

Please tell is nopie option is required here

It seems to build just fine without using nopie, so I think it can be removed here.

@Luciogi

Luciogi commented Aug 20, 2023

Copy link
Copy Markdown
Contributor Author

@Bnyro what about common/shlibs?

@Bnyro

Bnyro commented Aug 20, 2023

Copy link
Copy Markdown
Contributor

I'm not really familiar with that, sorry.

Looks fine to me, but you should not rely on me for that one.

@Luciogi

Luciogi commented Aug 20, 2023

Copy link
Copy Markdown
Contributor Author

can you ping someone to review ?

@Bnyro

Bnyro commented Aug 20, 2023

Copy link
Copy Markdown
Contributor

You can ask in the IRC room, it's linked in the Readme.

@Bnyro

Bnyro commented Aug 20, 2023

Copy link
Copy Markdown
Contributor

Comment thread srcpkgs/android-studio/template
@Luciogi

Luciogi commented Aug 20, 2023

Copy link
Copy Markdown
Contributor Author

@ahesford can you tell me about common/shlibs I dont understand which libs to include

@ahesford

Copy link
Copy Markdown
Member

No libraries from this package should be mentioned in 'common/shlibs`. Please remove your additions as well as the existing entries that you intended to replace.

@Luciogi

Luciogi commented Aug 20, 2023

Copy link
Copy Markdown
Contributor Author

@ahesford after deleting libs from common/shlibs breaks package build
https://pastebin.com/Ug5xKLHT
it seems now ncurses5 is required to build

would it possible to include compatibility for ncurses5 with current ncurses package (reference: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=ncurses5-compat-libs#n38)

Or I skip dependency check for these following files (temporarily until ncurse5 is packages )

./plugins/android-ndk/resources/lldb/lib/python3.10/lib-dynload/_curses.cpython-310-x86_64-linux-gnu.so
./plugins/android-ndk/resources/lldb/lib/python3.10/lib-dynload/_curses_panel.cpython-310-x86_64-linux-gnu.so

@ahesford

ahesford commented Aug 20, 2023

Copy link
Copy Markdown
Member

The inclusion of ncurses v5 compact libs has been discussed and rejected in the past. Skipping dependency checks for those two libraries is one possible workaround, but it may be better to remove those CPython extensions entirely (and anything that depends on them) because they will not be importable by Python.

@Luciogi Luciogi marked this pull request as ready for review August 21, 2023 06:16
@Luciogi

Luciogi commented Aug 21, 2023

Copy link
Copy Markdown
Contributor Author

@ahesford , Lint template test fails

edit: xlint template does not produce any error locally

@Bnyro

Bnyro commented Aug 21, 2023

Copy link
Copy Markdown
Contributor

authored by noreply email - you have to use a proper email for authoring the commit.

@Luciogi

Luciogi commented Aug 21, 2023

Copy link
Copy Markdown
Contributor Author

would simplelogin alias work here?

@Bnyro

Bnyro commented Aug 21, 2023

Copy link
Copy Markdown
Contributor

You should use an email you have and keep access to, I'm not familiar with Simplelogin.

@Luciogi

Luciogi commented Aug 21, 2023

Copy link
Copy Markdown
Contributor Author

lol same test was passed just before this commit

@Bnyro

Bnyro commented Aug 21, 2023

Copy link
Copy Markdown
Contributor

Only a network error, can be ignored.

@Luciogi

Luciogi commented Aug 21, 2023

Copy link
Copy Markdown
Contributor Author

Only a network error, can be ignored.

should I force push again
or reviewer will manually start checks ?

@Luciogi Luciogi changed the title android-studio: update to 2022.3.1.18 android-studio: update to 2022.3.1.18, orphan Aug 21, 2023
@classabbyamp classabbyamp merged commit bbebe57 into void-linux:master Sep 27, 2023
@Luciogi Luciogi deleted the android-studio branch March 6, 2024 16:04
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.

4 participants