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

[Issue Report]: libsodium issue on Manjaro Arm64 #6954

Closed
coreybruce opened this issue Feb 10, 2024 · 14 comments · Fixed by #6955
Closed

[Issue Report]: libsodium issue on Manjaro Arm64 #6954

coreybruce opened this issue Feb 10, 2024 · 14 comments · Fixed by #6955

Comments

@coreybruce
Copy link

Operating System

Other (please specify)

DevilutionX version

1.5.2

Describe

Hey just want to inform you that while you did fix the issue eith Libsodium on Manjaro x64 the issue still persists on Manjaro Arm64

./diablo: error while loading shared libraries: libsodium.so.23: cannot open shared object file: No such file or directory

I downloaded the Arm64 binary from your repo also

To Reproduce

Try and play game

Expected Behavior

Game runs

Additional context

No response

@AJenbo
Copy link
Member

AJenbo commented Feb 10, 2024

Please provide relevant info such as if libsodium is installed, if so what version.

Also consider using a package aimed at your distribution rather than the static build which depends on specific package version that may not be provided by your distribution, AUR should work for Manjaro as I understand it https://aur.archlinux.org/packages/devilutionx
This is especially relevant for rolling releases distros such as Manjaro.

We cannot maintain and test versions for every distribution out there, if you want to maintain support for Manjaro that is always an option.

@coreybruce
Copy link
Author

coreybruce commented Feb 10, 2024

Please provide relevant info such as if libsodium is installed, if so what version.

Also consider using a package aimed at your distribution rather then the genetic build which depends on specific package version that may not be provided by your distribution, AUR should work for Manjaro as I understand it https://aur.archlinux.org/packages/devilutionx

We cannot maintain and test versions for every distribution out there, if you want to maintain support for Manjaro that is always an option.

Yes it is installed and it is version 1.0.19-2

It works on Manjaro x64 also I don't think you saw even tho it is very suprising since you moderate the Github repos but 1.5.2 was made to fix the issue on Manjaro libsodium which previously reported

https://github.com/diasurgical/devilutionX/releases/tag/1.5.2

"Manjaro: Unable to locate libsodium"

The odd issue is that for some reason the issue persists on the Arm64 binary, it has been fixed on the x64 binary so it may need to be corrected on the Arm64 binary.

I also use the binaries on my AUR package for binaries https://aur.archlinux.org/packages/devilutionx-bin

I could even recompile it for Arm64 and provide the binary to you guys.

@AJenbo
Copy link
Member

AJenbo commented Feb 10, 2024

I don't think you saw even tho it is very suprising since you moderate the Github repos but 1.5.2 was made to fix the issue on Manjaro libsodium

What ever do you mean by this?

@coreybruce
Copy link
Author

coreybruce commented Feb 10, 2024

I was letting you know that they did a fix on this release specifically for Manjaro so it is supports distros like Manjaro

Also since I have provided the information that was needed you can remove the Insufficient information label

@qndel
Copy link
Member

qndel commented Feb 10, 2024

"The odd issue is that for some reason the issue persists on the Arm64 binary but not has been fixed on the x64 binary, it may need to be fixed." english pls

@AJenbo
Copy link
Member

AJenbo commented Feb 10, 2024

I don't think you saw even tho it is very suprising since you moderate the Github repos

Not only have I seen it, I wrote it. Maybe take the time to see who you are quoting before you start making accusations, assumptions and conclusions based on it: https://github.com/diasurgical/devilutionX/releases/tag/1.5.2

I was letting you know that they did a fix on this release specifically for Manjaro so it is supports distros like Manjaro

Who do you think "they" are? I was also the one who made the change to the x86_64 build script: #6938
Like we explained last time this embeds the libsodium library rather than rely on the system package. But configurations like that are build specific so that's why it doesn't affect other architectures.

I could even recompile it for Arm64 and provide the binary to you guys.

If it's built against whatever libraries are present on Manjaro at the time then it won't work on most other distros as the glibc version going to conflict with them. Besides the build is handled by the automated actions and it's prefered that way to avoid having potential mitm issues.

Also since I have provided the information that was needed you can remove the Insufficient information label

I'll get to it. Maybe you can add it to the original post while you wait.

@coreybruce
Copy link
Author

Not only have I seen it, I wrote it. Maybe take the time to see who you are quoting before you start making accusations, assumptions and conclusions based on it: https://github.com/diasurgical/devilutionX/releases/tag/1.5.2

why are you getting aggressive and assuming I am making accusation here lol

Who do you think "they" are? I was also the one who made the change to the x86_64 build script: #6938

I was referring to the devilutionX in general here and if you were the one who did the fix then great

Like we explained last time this embeds the libsodium library rather than rely on the system package. But configurations like that are build specific so that's why it doesn't affect other architectures.

Yes I know that, I was just letting you know that the fix you did for x64 in the build script will also need to be applied to the Arm64 binary when compiling, that's all I am informing you about.

@coreybruce
Copy link
Author

"The odd issue is that for some reason the issue persists on the Arm64 binary but not has been fixed on the x64 binary, it may need to be fixed." english pls

Fixed it

@AJenbo
Copy link
Member

AJenbo commented Feb 10, 2024

why are you ... assuming I am making accusation here lol

Your message implies doubt about whether I saw something despite my role in moderating the GitHub repositories, suggesting I should have seen it. Where I come from that's called an accusation.

why are you getting aggressive

You also chose to use an intensifiers in that statement. You not even realizing that you are quoting me back to me gave me the impression that this was going to be a repeat of our last which was rather unpleasant.

Yes I know that, I was just letting you know that the fix you did for x64 in the build script will also need to be applied to the Arm64 binary when compiling, that's all I am informing you about.

This contradicts you previous statement "The odd issue is that for some reason the issue persists on the Arm64 binary". But I'm glad that you are now aware of it.

@coreybruce
Copy link
Author

coreybruce commented Feb 10, 2024

Wow ok how about we get back on topic and stop being aggressive and making aggressive assumptions, I never implied or had doubts, I was just informing about the issue etc in case you didn't know also no I didn't know you wrote the post but you were acting like it wasn't supported despite it does and the fix for Manjaro was there but not completely fixed sadly

Getting back to the topic at hand I just want to help fix the issue but attacking me isn't going to solve anything, I am your friend not your enemy

lets just stop fighting please 😃

@AJenbo
Copy link
Member

AJenbo commented Feb 10, 2024

I never implied or had doubts

I don't know how else to read this "I don't think you saw even tho it is very suprising"?

you were acting like it wasn't supported despite it does and the fix for Manjaro was there but not completely fixed sadly

I think you misunderstood something then. It's a build time option, not something that you can fully "fix", it depends on how you run the build process for each binary.

I have updated the release nots to hopefully make it less confusing: "Linux: Statically link BZIP2 and libsodium on x86_64"

Getting back to the topic at hand I just want to help fix the issue but attacking me isn't going to solve anything, I am your friend not your enemy

You are not the only one feeling attacked here.

@coreybruce
Copy link
Author

coreybruce commented Feb 10, 2024

I don't know how else to read this "I don't think you saw even tho it is very suprising"?

Well I am sorry if you feel like I was doubting you but I really wasn't, I was just unsure oh who did what on this project as I don't know all the developers or jobs well.

I think you miss understood something then. It's a build time option, not something that you can fully "fix", it depends on how you run the build process for each binary.

I assume we would need to rebuild the binary for Arm64 to fix it

I have updated the release notes to hopefully make it less confusing: "Linux: Statically link BZIP2 and libsodium on x86_64"

Alright, does Arm64 do the same thing?

You are not the only one feeling attacked here.

I'm sorry if you feel attacked, that wasn't my attention and I just wanted to report the issue and help fix it but you have to admit you did come in here guns blazing a little bit just because I didn't know you wrote the release post and I don't know all the developers here. I do wish to be good friends with developers and help any way I can so I am sorry if I did offend you.

Is there anything else I can help you guys with at all? do you guys need me to be a tester at all?

@AJenbo
Copy link
Member

AJenbo commented Feb 10, 2024

I don't think we will be re-releasing the 1.5.2 Linux Aarch64 build since it will reset the download counter and it has already been a week since the release. It would also cause the hash to change which might cause issues down stream.

If you want to prevent issues like that it would be good if you can test the test builds prior to a release: https://github.com/diasurgical/devilutionX?tab=readme-ov-file#test-builds

I felt defensive because it seemed like you were making unnecessary personal comments and denying it when I pointed it out.

@coreybruce
Copy link
Author

coreybruce commented Feb 11, 2024

I don't think we will be re-releasing the 1.5.2 Linux Aarch64 build since it will reset the download counter and it has already been a week since the release. It would also cause the hash to change which might cause issues down stream.

Ah ok that's understandable 👍

If you want to prevent issues like that it would be good if you can test the test builds prior to a release: https://github.com/diasurgical/devilutionX?tab=readme-ov-file#test-builds

Yeah no problem I can do that

I felt defensive because it seemed like you were making unnecessary personal comments and denying it when I pointed it out.

Oh I'm sorry but that wasn't my intention at all or what I was trying to do.

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 a pull request may close this issue.

3 participants