Package Details: nzbhydra2-nojava-bin 7.12.3-1

Git Clone URL: https://aur.archlinux.org/nzbhydra2-nojava-bin.git (read-only, click to copy)
Package Base: nzbhydra2-nojava-bin
Description: Search aggregator for newznab and torznab indexers.
Upstream URL: https://github.com/theotherp/nzbhydra2
Licenses: Apache
Conflicts: nzbhydra2
Provides: nzbhydra2
Submitter: gee
Maintainer: gee (jkhsjdhjs)
Last Packager: jkhsjdhjs
Votes: 0
Popularity: 0.000000
First Submitted: 2023-01-23 12:31 (UTC)
Last Updated: 2025-01-31 15:34 (UTC)

Dependencies (2)

Required by (23)

Sources (6)

Latest Comments

« First ‹ Previous 1 2 3 4 Next › Last »

gee commented on 2023-01-27 00:20 (UTC)

I still get the "Outdated wrappers detected" popup on the nzbhydra2 webpage, so it looks no different to before. It's possible I still have some old file left on my system that messes things up but I don't see where.

Thanks!

jkhsjdhjs commented on 2023-01-26 23:20 (UTC) (edited on 2023-01-26 23:22 (UTC) by jkhsjdhjs)

Did you also adjust nzbhydra2.sh accordingly? If you rename the wrapper you also have to specify the new name there. Renaming the wrapper to something other than

NZBHydra2.exe, NZBHydra2 Console.exe, nzbhydra2, nzbhydra2wrapper.py, nzbhydra2wrapperPy3.py

works for me.

EDIT: I just noticed you pushed the changes. So yeah, you did adjust nzbhydra2.sh accordingly, and it does work for me. Maybe we have different expectations. What is happenng for you after the name change?

gee commented on 2023-01-26 20:38 (UTC)

@jkhsjdhjs I've tried renaming the 3py file but it didn't help, just in case I tried renaming the nzbhydra2 file back to nzbhydra2.sh since the short one is in the list, but no go either.

It only seems to check for these: "NZBHydra2.exe", "NZBHydra2 Console.exe", "nzbhydra2", "nzbhydra2wrapper.py", "nzbhydra2wrapperPy3.py". I'm not sure what else to try.

Thank you!

jkhsjdhjs commented on 2023-01-26 15:42 (UTC)

@gee Yeah, the link you posted is exactly the issue. The hashes are stored within the executable, so there's no way to change them. With nzbhydra2-bin (the java version), nzbhydra2 is unable to find the wrapper (because the jar file resides in a different directory), so it doesn't show the notice but instead just logs an error on startup:

2023-01-26 16:31:05.032 ERROR --- [HT-Problem detecto] o.n.p.OutdatedWrapperDetector            : Didn't find any of the expected wrapper files (/usr/share/java/nzbhydra2) in folder NZBHydra2.exe, NZBHydra2 Console.exe, nzbhydra2, nzbhydra2wrapper.py, nzbhydra2wrapperPy3.py

In the package that is built from source we remove that check entirely so it doesn't even log an error (https://aur.archlinux.org/cgit/aur.git/tree/disable-update-checks.patch?h=nzbhydra2), but if you're fine with that error being logged you can simply install the wrapper with a different name.

I also thought about removing the wrapper entirely, but it's still useful for restoring backups: https://github.com/theotherp/nzbhydra2/blob/master/other/wrapper/nzbhydra2wrapperPy3.py#L223

gee commented on 2023-01-26 08:39 (UTC) (edited on 2023-01-26 08:39 (UTC) by gee)

@jkhsjdhjs sorry to bother you and just in case you know already, I'm trying to remove the warning that the scripts are outdated. I'm guessing the issue is https://github.com/theotherp/nzbhydra2/blob/7b7949c32209358a8dc53cd2c9eba0963e1b16ab/core/src/main/java/org/nzbhydra/problemdetection/OutdatedWrapperDetector.java#L69 the hashes don't match the expected ones after the patches but I fear the json is embeded in the executable so I cannot change that one. Do you already know another place to disable that check?

Thanks!

jkhsjdhjs commented on 2023-01-24 01:19 (UTC)

@gee The nzbhydra2 binary is simply a compiled version of the wrapper script. While using it would allow us to drop the dependency on python, we wouldn’t be able to modify the script anymore, thus preventing us from installing the files to the directories they belong to according to the packaging guidelines. Also, the binary is only compiled for x86_64, so using it would break aarch64 support.

gee commented on 2023-01-23 23:39 (UTC)

@jkhsjdhjs Awesome, thank you very much!

How did you know the nzbhydra2 binary was not needed?

jkhsjdhjs commented on 2023-01-23 15:49 (UTC)

@gee alright, should work as expected now.

gee commented on 2023-01-23 12:32 (UTC)

@jkhsjdhjs alright, renamed as suggested.

jkhsjdhjs commented on 2023-01-23 12:14 (UTC) (edited on 2023-01-23 12:21 (UTC) by jkhsjdhjs)

@gee I think -compiled is fine, but I'd suggest you change it to -compiled-bin. First, because the AUR package guidelines require that, but also because there may be a package compiling the binaries from source someday, which it would make sense to name nzbhydra2-compiled.

As for the core file, it is the actual executable for nzbhydra2, the java version simply has a core.jar in its place.

EDIT: Actually, I think -nojava-bin would be even better, as the jar files for the other package are compiled as well, so -compiled is kinda ambiguous.

« First ‹ Previous 1 2 3 4 Next › Last »