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

minosoft: init at 7ba4569 #244566

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

CutestNekoAqua
Copy link
Contributor

Description of changes

From the project's GitLab: "Minosoft is an open source minecraft client, written from scratch in kotlin (and java). It aims to bring more functionality and stability."

Resolves #237956

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.11 Release Notes (or backporting 23.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.


let
pname = "minosoft";
version = "7ba4569"; # there are no tags yet, so lets version this using tags for now
Copy link
Member

Choose a reason for hiding this comment

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

In such case, you must use

Suggested change
version = "7ba4569"; # there are no tags yet, so lets version this using tags for now
version = "unstable-YYYY-MM-DD";

where the YYYY-MM-DD is the date of the commit you are picking.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will do

pkgs/games/minosoft/default.nix Outdated Show resolved Hide resolved
buildPhase = ''
export GRADLE_USER_HOME=$(mktemp -d)
# https://github.com/gradle/gradle/issues/4426
${lib.optionalString stdenv.isDarwin "export TERM=dumb"}
Copy link
Member

Choose a reason for hiding this comment

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

Can this be set unconditionally?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will evaluate what makes more sense.


nativeBuildInputs = [ gradle perl makeWrapper copyDesktopItems ];

desktopItems = [ desktopItem ];
Copy link
Member

Choose a reason for hiding this comment

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

Write the desktopItem directly here.

pkgs/games/minosoft/default.nix Show resolved Hide resolved
pkgs/games/minosoft/default.nix Outdated Show resolved Hide resolved
maintainers = with maintainers; [ aprl ];
platforms = platforms.all;
# https://github.com/NixOS/nixpkgs/pull/99885#issuecomment-740065005
#broken = stdenv.isDarwin;
Copy link
Member

Choose a reason for hiding this comment

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

??

@CutestNekoAqua
Copy link
Contributor Author

Note for future reviewers: This PR is not ready yet, as why its marked WIP. There is currently a issue with the gradle deps not working correctly because of some issues with the perl script. Currently evaluating on a better way to replace the old perl script from Minestom and replacing it either with a updated version of the old gradle2nix which got abandoned, or a even newer way of a new and improved script for the current perl hack

@wegank wegank added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Mar 19, 2024
@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label May 3, 2024
@wegank wegank added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jul 4, 2024
@chayleaf
Copy link
Contributor

#272380 got merged, so you can give this a second shot now! See https://github.com/NixOS/nixpkgs/blob/master/doc/languages-frameworks/gradle.section.md

@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Package request: Minosoft
5 participants