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
fedpkg: remove unneeded patch #51229
Conversation
@GrahamcOfBorg build python3.pkgs.fedpkg python2.pkgs.fedpkg |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: python3.pkgs.fedpkg, python2.pkgs.fedpkg Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python2.pkgs.fedpkg The following builds were skipped because they don't evaluate on x86_64-linux: python3.pkgs.fedpkg Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python2.pkgs.fedpkg The following builds were skipped because they don't evaluate on aarch64-linux: python3.pkgs.fedpkg Partial log (click to expand)
|
@marsam Can you remove |
The patch might still be needed: $ exa -T result/
result
├── bin
│ ├── fedpkg
│ └── fedpkg-stage
├── lib
│ └── python2.7
│ └── site-packages
│ ├── etc
│ │ ├── bash_completion.d
│ │ │ └── fedpkg.bash
│ │ └── rpkg
│ │ ├── fedpkg-stage.conf
│ │ └── fedpkg.conf
│ ├── fedpkg
│ │ ├── __init__.py
│ │ ├── __init__.pyc
│ │ ├── __main__.py
│ │ ├── __main__.pyc
│ │ ├── cli.py
│ │ ├── cli.pyc
│ │ ├── lookaside.py
│ │ └── lookaside.pyc
│ ├── fedpkg-1.29.dist-info
│ │ ├── COPYING
│ │ ├── INSTALLER
│ │ ├── METADATA
│ │ ├── RECORD
│ │ ├── top_level.txt
│ │ └── WHEEL
│ └── usr
│ └── share
│ └── zsh
│ └── site-functions
│ └── _fedpkg
└── nix-support
└── propagated-build-inputs |
Perhaps an update and a slightly different patch would be preferable https://pagure.io/fedpkg/blob/master/f/setup.py#_51 |
736dc63
to
81071b4
Compare
I've updated the patch
|
81071b4
to
9924be2
Compare
@marsam Would you mind doing other cleanups for this? |
sure, do you have anything in mind? |
Yep, just wanted to ask before I started reviewing again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this a tool and not meant to be used as an importable module.
So we should use buildPythonApplication
(similar to what was done in #50863) and move it to a better home in nixpkgs.
Also, all that cleanup I think an update would be cool. |
9924be2
to
2256273
Compare
I've updated fedpkg with Also, I did try to update it, but that it's going to take a bit of time: 3c1823c9838, because they don't quite follow the standard python packaging. I think that can go in another PR. |
👍
I agree that a separate pr for that would be appropriate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Builds and executes locally. Also I can clone a repo anonymously.
Though I had to specify the path to a modified config with the new urls.
Motivation for this change
This patch is no longer used
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)