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

guymager: init at 0.8.13 #278529

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

guymager: init at 0.8.13 #278529

wants to merge 2 commits into from

Conversation

D3vil0p3r
Copy link
Contributor

@D3vil0p3r D3vil0p3r commented Jan 3, 2024

To be checked in #81418

Depends on:

Description of changes

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • 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/)
  • 24.05 Release Notes (or backporting 23.05 and 23.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
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@D3vil0p3r D3vil0p3r mentioned this pull request Jan 21, 2024
13 tasks
@D3vil0p3r D3vil0p3r marked this pull request as ready for review January 29, 2024 13:49
@D3vil0p3r D3vil0p3r marked this pull request as draft January 29, 2024 13:49
@D3vil0p3r D3vil0p3r marked this pull request as ready for review February 11, 2024 01:51
@D3vil0p3r D3vil0p3r marked this pull request as draft February 11, 2024 02:08
pkgs/by-name/gu/guymager/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/gu/guymager/package.nix Outdated Show resolved Hide resolved
@D3vil0p3r
Copy link
Contributor Author

@ofborg eval

Copy link
Member

@emilytrau emilytrau left a comment

Choose a reason for hiding this comment

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

  • Needs split into a commit for each package
  • Needs rebase

pkgs/by-name/gu/guymager/package.nix Outdated Show resolved Hide resolved
@D3vil0p3r
Copy link
Contributor Author

  • Needs split into a commit for each package
  • Needs rebase

@emilytrau fixed

@sarcasticadmin
Copy link
Member

Result of nixpkgs-review pr 278529 run on x86_64-linux 1

2 packages built:
  • guymager
  • libguytools

pkgs/by-name/li/libguytools/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/li/libguytools/package.nix Outdated Show resolved Hide resolved
pkgs/by-name/li/libguytools/package.nix Outdated Show resolved Hide resolved
enableParallelBuilding = true;

nativeBuildInputs = [ libsForQt5.qmake libsForQt5.wrapQtAppsHook ];
dontWrapQtApps = true;
Copy link
Member

Choose a reason for hiding this comment

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

why?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eclairevoyant If I remove that, I get the following error:

Running phase: qtPreHook
Error: wrapQtAppsHook is not used, and dontWrapQtApps is not set.
error: builder for '/nix/store/fncpc8jyf7b1h08yipf25pkmls71pxhn-libguytools-2.1.0.drv' failed with exit code 1;
       last 10 log lines:
       > calling 'envTargetTargetHook' function hook 'make_glib_find_gsettings_schemas' /nix/store/qqzz17mjs51kpfx9faw98f35r7ffwqng-xcb-util-renderutil-0.3.10-dev
       > calling 'envTargetTargetHook' function hook 'make_glib_find_gsettings_schemas' /nix/store/7rd4b08axmpg5nnys8kkjbmj7cxlkdlc-xcb-util-renderutil-0.3.10
       > calling 'envTargetTargetHook' function hook 'make_glib_find_gsettings_schemas' /nix/store/x3l69111mbfd38dd8q59hqv94vz6d6jn-xcb-util-wm-0.4.2-dev
       > calling 'envTargetTargetHook' function hook 'make_glib_find_gsettings_schemas' /nix/store/fiwwkqgfgb4g2f63iycpsx2h5amk2jdv-xcb-util-wm-0.4.2
       > calling 'envTargetTargetHook' function hook 'make_glib_find_gsettings_schemas' /nix/store/h5j6ggf0g230bavqarsvyi23zw789fv3-libglvnd-1.7.0-dev
       > calling 'envTargetTargetHook' function hook 'make_glib_find_gsettings_schemas' /nix/store/d6gvgzzifggrb7fh1v0yi8bvrdlwhpqa-libglvnd-1.7.0
       > calling 'envTargetTargetHook' function hook 'make_glib_find_gsettings_schemas' /nix/store/hjspqdvym6v553582mxp2a3f84p4aqyr-qtbase-5.15.14-bin
       > calling 'envTargetTargetHook' function hook 'make_glib_find_gsettings_schemas' /nix/store/9wba9diw8b199hfzcv8w99wzma66dzm1-qtbase-5.15.14
       > Running phase: qtPreHook
       > Error: wrapQtAppsHook is not used, and dontWrapQtApps is not set.
       For full logs, run 'nix log /nix/store/fncpc8jyf7b1h08yipf25pkmls71pxhn-libguytools-2.1.0.drv'.

I can also use only:

nativeBuildInputs = [ libsForQt5.qmake libsForQt5.wrapQtAppsHook ];

and remove dontWrapQtApps. What should I use?

pkgs/by-name/gu/guymager/package.nix Outdated Show resolved Hide resolved
, parted
, udev
, libewf-legacy
, libbfio
Copy link
Member

Choose a reason for hiding this comment

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

this doesn't exist in the nixpkgs rev you're using, please rebase on current master

Copy link
Contributor Author

@D3vil0p3r D3vil0p3r Jul 12, 2024

Choose a reason for hiding this comment

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

Just for confirmation, the right commands to do this are:

git clone https://github.com/D3vil0p3r/nixpkgs -b patch-9
git remote add upstream https://github.com/NixOS/nixpkgs.git
git fetch upstream
git rebase upstream/master
git push -f origin patch-9

?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

After git rebase upstream/master, there are a lot of files that need to fix conflicts...

...
pkgs/top-level/release-lib.nix: needs merge
pkgs/top-level/release-python.nix: needs merge
pkgs/top-level/release-r.nix: needs merge
pkgs/top-level/release-small.nix: needs merge
pkgs/top-level/release.nix: needs merge
pkgs/top-level/ruby-packages.nix: needs merge
pkgs/top-level/splice.nix: needs merge
pkgs/top-level/stage.nix: needs merge
pkgs/top-level/unixtools.nix: needs merge
pkgs/top-level/wine-packages.nix: needs merge
You must edit all merge conflicts and then
mark them as resolved using git add

Probably it is better to close this PR and open a new one?

Copy link
Member

Choose a reason for hiding this comment

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

GitHub interface is sub subpar.

You can use the bulldozer method:

  • Save guymager/package.nix to a safe location (say ~/backups/)
  • create a local patch-9 branch
  • get back guymager/package.nix
  • commit and force-push

Copy link
Member

@eclairevoyant eclairevoyant Jul 12, 2024

Choose a reason for hiding this comment

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

@D3vil0p3r you should already have a local clone of nixpkgs if you're creating PRs, as rebasing is often necessary.
Ensure you do not use a shallow or treeless clone; use a blobless clone if you need to save time (--filter=blob:none)

For rebasing itself, I would use git rebase -i as you can see in advance what commits will get picked and adjust the list as needed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So, if I understood well, I should blobless clone the nixpkgs repo of my patch-9:

git clone https://github.com/D3vil0p3r/nixpkgs -b patch-9 --filter=blob:none
cd nixpkgs
git rebase -i

but I get only noop. Should I do additional things? Sorry for wasting your time guys.

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

7 participants