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

Jackett: Update to v0.20.3521 #5597

Merged
merged 6 commits into from
Mar 7, 2023
Merged

Conversation

mreid-tt
Copy link
Contributor

@mreid-tt mreid-tt commented Feb 10, 2023

Description

This PR includes the following:

  1. Update Jackett to v0.20.3521
  2. Remove unsupported ARMv7 archs from dotNET builds

Follow up of #5574
Fixes #5130 (partially)

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • Package update

@mreid-tt
Copy link
Contributor Author

mreid-tt commented Feb 10, 2023

@hgy59, I've dug into the current Jackett distributions which now seem to include dotNET 6 core in their non-mono distributions. The various Makefile configurations were updated and I removed our inclusion of dotNET 6 so it looks closer to a regular *arr package. I think it's ready to go but may need some more testing on other archs (as I only tested on x64).

Once there are no issues and the package is published, I would recommend un-publishing v0.20.1171-15 from the repository to remove the incompatible combination of ARMv7 archs and DSM6.

@mreid-tt mreid-tt requested a review from hgy59 February 10, 2023 14:12
@mreid-tt mreid-tt changed the title Jackett: Update to v0.20.3017 Jackett: Update to v0.20.3127 Feb 16, 2023
@mreid-tt mreid-tt requested a review from publicarray February 19, 2023 11:10
@mreid-tt
Copy link
Contributor Author

mreid-tt commented Feb 20, 2023

@hgy59, I've done some basic testing under DSM7 as follows:

All in all, it seems good to merge.

@mreid-tt mreid-tt requested a review from hgy59 February 22, 2023 11:00
@mreid-tt
Copy link
Contributor Author

@hgy59, is this good to merge?

@mreid-tt mreid-tt changed the title Jackett: Update to v0.20.3127 Jackett: Update to v0.20.3521 Mar 7, 2023
@mreid-tt
Copy link
Contributor Author

mreid-tt commented Mar 7, 2023

@publicarray, would appreciate a review so that this can be merged.

Copy link
Contributor

@hgy59 hgy59 left a comment

Choose a reason for hiding this comment

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

LGTM

@mreid-tt mreid-tt merged commit ace9f26 into SynoCommunity:master Mar 7, 2023
@mreid-tt mreid-tt deleted the jackett-patch branch March 7, 2023 22:49
@hgy59 hgy59 added status/published Published and activated (may take up to 48h until visible in DSM package manager) and removed status/to-publish labels Mar 7, 2023
AlexPresso pushed a commit to AlexPresso/spksrc that referenced this pull request Jan 30, 2025
* Update Jackett to v0.20.3521
* Remove unsupported ARMv7 archs from dotNET builds
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/published Published and activated (may take up to 48h until visible in DSM package manager)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RADARR SONAR and JACKETT
2 participants