Skip to content

ccemux: mark broken (download fails hash validation)#272193

Closed
ghost wants to merge 1 commit intomasterfrom
unknown repository
Closed

ccemux: mark broken (download fails hash validation)#272193
ghost wants to merge 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Dec 5, 2023

This package's FOD fails hash validation.

@ghost ghost marked this pull request as ready for review December 5, 2023 06:07
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Dec 5, 2023
sourceProvenance = with sourceTypes; [ binaryBytecode ];
license = licenses.mit;
maintainers = with maintainers; [ CrazedProgrammer viluon ];
broken = true; # download of CCEmuX-cct.jar fails hash validation
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can we update hash ?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Since I don't use this software I am not qualified to decide if that's safe.

Usually the best course here is to mark the package as broken and if nobody steps up, let the package get garbage-collected. If somebody does step up to audit the mysteriously-changed binary jarfile, and wants to take the reputational risk upon themselves, more power to them!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

maybe @viluon can have a look ( last committer ), otherwise I agree with your assessment

Copy link
Copy Markdown
Contributor

@superherointj superherointj Jan 20, 2024

Choose a reason for hiding this comment

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

I agree with @amjoseph-nixpkgs . My nitpick is to open a tracking issue and reference it. So discussions of fixing that package happens in that issue. And add the day the breakage is being reported.

Copy link
Copy Markdown

@MCJack123 MCJack123 Jan 21, 2024

Choose a reason for hiding this comment

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

Upstream doesn't do version control on binaries AFAIK. The maintainer will have to be on top of version bumps to avoid this issue from happening. I don't use Nix, so I don't know if this is an option (probably isn't), but it may also be necessary to skip checking hashes to avoid breakages between updates.

Disregard this, I've been corrected.

Copy link
Copy Markdown
Contributor

@viluon viluon Jan 21, 2024

Choose a reason for hiding this comment

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

Nah, see CCEmuX/CCEmuX#167. Thanks @kirillrdy for the mention.

@viluon
Copy link
Copy Markdown
Contributor

viluon commented Jan 21, 2024

Per @SquidDev's response, it should be safe to just update the hash, although the package will remain flaky.

I agree that a full Nix build would be better, maybe the original packager @CrazedProgrammer has tried this in the past? I've had little luck with Gradle builds in Nix myself, but maybe this is a simple enough project that it could work.

@ghost ghost closed this Jan 23, 2024
@ghost ghost deleted the pr/broken/ccemux branch January 23, 2024 06:47
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants