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

rofi-blezz: init at unstable-2022-09-07 #221729

Merged
merged 2 commits into from
May 22, 2024
Merged

Conversation

JohnJohnstone
Copy link
Contributor

@JohnJohnstone JohnJohnstone commented Mar 17, 2023

Description of changes

Adds rofi-blezz

Things done

Tested on sway and i3

  • Built on platform(s)
    • [ X] x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • [X ] 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
  • [ X] Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.05 Release Notes (or backporting 22.11 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
  • [ X] Fits CONTRIBUTING.md.

Copy link
Member

@ocfox ocfox left a comment

Choose a reason for hiding this comment

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

Also squash commit, add maintainer and package init.

pkgs/applications/misc/rofi-blezz/default.nix Outdated Show resolved Hide resolved
@programmerHaaks
Copy link

What's missing for this to get merged? Just the squash?

@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 14, 2024
@isabelroses
Copy link
Member

isabelroses commented May 14, 2024

The title should be changed to rofi-blezz: init at unstable-2022-09-07. The all the commits except the commit titled maintainers: add johnjohnstone should also be squashed into a commit titled rofi-blezz: init at unstable-2022-09-07. The package should also be moved into the by-name standard, https://github.com/NixOS/nixpkgs/blob/master/pkgs/by-name/README.md.

@JohnJohnstone JohnJohnstone changed the title rofi-blezz: Init Package and Maintainer rofi-blezz: init at unstable-2022-09-07 May 14, 2024
@JohnJohnstone
Copy link
Contributor Author

Not sure how to squash my commits, I have done it before but likely due to error on my part I have other people commits in between my own and since this is a few years old I cant even find my commit in the tool i use for git. I could just make a new pull request? Any help would be much appreciated

@isabelroses
Copy link
Member

Not sure how to squash my commits, I have done it before but likely due to error on my part I have other people commits in between my own and since this is a few years old I cant even find my commit in the tool i use for git. I could just make a new pull request? Any help would be much appreciated

To squash your commits you can type git rebase -i 2903f85, this will open up your editor with a collection of your last serval commits. Leave the commit titled rofi-blezz: Init at unstable-2022-09-07 as pick rofi-blezz: Init at unstable-2022-09-07 whilst changing the other commits to say s commit-name. Then you will have to force push to this branch git push -f.

@JohnJohnstone
Copy link
Contributor Author

Not sure how to squash my commits, I have done it before but likely due to error on my part I have other people commits in between my own and since this is a few years old I cant even find my commit in the tool i use for git. I could just make a new pull request? Any help would be much appreciated

To squash your commits you can type git rebase -i 2903f85, this will open up your editor with a collection of your last serval commits. Leave the commit titled rofi-blezz: Init at unstable-2022-09-07 as pick rofi-blezz: Init at unstable-2022-09-07 whilst changing the other commits to say s commit-name. Then you will have to force push to this branch git push -f.

Thank You, Commit History should be fixed now.

pkgs/by-name/ro/rofi-blezz/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/ro/rofi-blezz/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/ro/rofi-blezz/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/ro/rofi-blezz/package.nix Outdated Show resolved Hide resolved
@JohnJohnstone JohnJohnstone force-pushed the master branch 2 times, most recently from 717736a to 7278765 Compare May 15, 2024 11:20
nativeBuildInputs = [
autoreconfHook
pkg-config
wrapGAppsHook3
Copy link
Member

Choose a reason for hiding this comment

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

wrappers are wrapping executables not libraries, so what does wrapGAppsHook3 do here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah it is not required. Based my package on rofi-top that uses it. I have removed the wrapper and rofi-blezz works as expected.

@Aleksanaa Aleksanaa merged commit 9217de2 into NixOS:master May 22, 2024
25 checks passed
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.

None yet

6 participants