8000 Multi-arch rules by r58Playz · Pull Request #680 · PolyMC/PolyMC · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Multi-arch rules #680

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

Closed
wants to merge 39 commits into from
Closed

Multi-arch rules #680

wants to merge 39 commits into from

Conversation

r58Playz
Copy link
@r58Playz r58Playz commented May 30, 2022

Just #514 but updated to newest develop.
Works with PolyMC/meta-scripts#16 to provide multi-arch support.

edit: Fixes #382
edit 2: Fixes #765 but not really, just a workaround fixed already

@d-513
Copy link
Contributor
d-513 commented May 30, 2022

Please squash my commits. Dealing with GHA is painful

@timoreo22
Copy link
Contributor

There also is a few other issues with this pr like

  1. a random java file got deleted ? that's the one failing CI
  2. there is still a vdf parser lib, probably shouldn't be in this pr
  3. there is also a random file named share in there

All of that should probably be deleted and squashed out, otherwise the code looks fine

@d-513
Copy link
Contributor
d-513 commented May 30, 2022

There also is a few other issues with this pr like

1. a random java file got deleted ? that's the one failing CI

2. there is still a vdf parser lib, probably shouldn't be in this pr

3. there is also a random file named share in there

All of that should probably be deleted and squashed out, otherwise the code looks fine

yea definietly. The share is actually a symlink I did for CLION to not break, but it can be removed

@r58Playz
Copy link
Author
r58Playz commented Jun 2, 2022

There also is a few other issues with this pr like

1. a random java file got deleted ? that's the one failing CI

2. there is still a vdf parser lib, probably shouldn't be in this pr

3. there is also a random file named share in there

All of that should probably be deleted and squashed out, otherwise the code looks fine

All fixed and updated to newest upstream

Copy link
Contributor
@d-513 d-513 left a comment

Choose a reason for hiding this comment

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

reviewing mostly own pr be like

@r58Playz r58Playz requested a review from d-513 June 2, 2022 20:11
@flowln flowln added enhancement New feature or request meta Stuff with meta. macOS macOS-related stuff and removed macOS macOS-related stuff labels Jun 6, 2022
@Scrumplex
Copy link
Contributor

@r58Playz how does this fix #765?

@DioEgizio
Copy link
Contributor

@r58Playz how does this fix #765?

image

@Scrumplex
Copy link
Contributor

I wouldn't call that a fix tbh.

@DioEgizio
Copy link
Contributor

Yeah it's more a workaround

@d-513
Copy link
Contributor
d-513 commented Jun 9, 2022

But why download useless libraries tho? Downloading arm on x86-64 doesn’t make sense

@Scrumplex
Copy link
10000 Contributor

But why download useless libraries tho? Downloading arm on x86-64 doesn’t make sense

Because you can't run arm natives with a x86-64 JRE. The point is, that you can't mix CPU architectures for a running application. If the user uses a x86-64 JRE, then they need x86-64 natives.

r58Playz and others added 2 commits June 9, 2022 13:33
Co-authored-by: Sefa Eyeoglu <contact@scrumplex.net>
Co-authored-by: Sefa Eyeoglu <contact@scrumplex.net>
@r58Playz r58Playz requested a review from Scrumplex July 1, 2022 22:22
r58Playz added 6 commits July 9, 2022 10:00
Signed-off-by: Toshit Chawda <r58playz@gmail.com>
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: 02b8588

Signed-off-by: r58Playz <toshit307@gmail.com>
…github.com>

I, r58Playz <45221816+r58Playz@users.noreply.github.com>, hereby add my Signed-off-by to this commit: bd38a7c
I, r58Playz <45221816+r58Playz@users.noreply.github.com>, hereby add my Signed-off-by to this commit: 9b6a878
I, r58Playz <45221816+r58Playz@users.noreply.github.com>, hereby add my Signed-off-by to this commit: cae505a

Signed-off-by: r58Playz <45221816+r58Playz@users.noreply.github.com>
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: ff122bb
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: e4a2972
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: a207dff
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: e5ca5ff
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: 000f892
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: a7140b3
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: ea69d59
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: 4d844e1
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: d0c818d

Signed-off-by: r58Playz <toshit307@gmail.com>
I, r58Playz <r58playz@gmail.com>, hereby add my Signed-off-by to this commit: e0068b6
I, r58Playz <r58playz@gmail.com>, hereby add my Signed-off-by to this commit: b67c7a2
I, r58Playz <r58playz@gmail.com>, hereby add my Signed-off-by to this commit: fc548e8
I, r58Playz <r58playz@gmail.com>, hereby add my Signed-off-by to this commit: ae15f6c

Signed-off-by: r58Playz <r58playz@gmail.com>
@DioEgizio
Copy link
Contributor

edit 2: Fixes #765 but not really, just a workaround

that issue was fixed ages ago, just so you know

r58Playz and others added 14 commits July 9, 2022 12:34
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: 02b8588

Signed-off-by: r58Playz <toshit307@gmail.com>
…oreply.github.com>

I, r58Playz <45221816+r58Playz@users.noreply.github.com>, hereby add my Signed-off-by to this commit: bd38a7c
I, r58Playz <45221816+r58Playz@users.noreply.github.com>, hereby add my Signed-off-by to this commit: 9b6a878
I, r58Playz <45221816+r58Playz@users.noreply.github.com>, hereby add my Signed-off-by to this commit: cae505a

Signed-off-by: r58Playz <45221816+r58Playz@users.noreply.github.com>
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: ff122bb
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: e4a2972
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: a207dff
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: e5ca5ff
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: 000f892
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: a7140b3
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: ea69d59
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: 4d844e1
I, r58Playz <toshit307@gmail.com>, hereby add my Signed-off-by to this commit: d0c818d

Signed-off-by: r58Playz <toshit307@gmail.com>
I, r58Playz <r58playz@gmail.com>, hereby add my Signed-off-by to this commit: 864ae88
I, r58Playz <r58playz@gmail.com>, hereby add my Signed-off-by to this commit: e825f0b
I, r58Playz <r58playz@gmail.com>, hereby add my Signed-off-by to this commit: 45ad5aa
I, r58Playz <r58playz@gmail.com>, hereby add my Signed-off-by to this commit: 027d634

Signed-off-by: r58Playz <r58playz@gmail.com>
Signed-off-by: r58Playz <r58playz@gmail.com>
Signed-off-by: r58Playz <r58playz@gmail.com>
Signed-off-by: r58Playz <r58playz@gmail.com>
Signed-off-by: r58Playz <r58playz@gmail.com>
Signed-off-by: r58Playz <r58playz@gmail.com>
2975a20

Signed-off-by: r58Playz <r58playz@gmail.com>
@Scrumplex Scrumplex mentioned this pull request Aug 8, 2022
3 tasks
@r58Playz
Copy link
Author
r58Playz commented Aug 8, 2022

#1033

@r58Playz r58Playz closed this Aug 8, 2022
@timoreo22
Copy link
Contributor

Get scrumped

@ryanccn
Copy link
Contributor
ryanccn commented Aug 9, 2022

Scrumped

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request meta Stuff with meta.
Projects
None yet
0