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

eclean : deal with invalid binpkgs #28

Closed
wants to merge 4 commits into from
Closed

eclean : deal with invalid binpkgs #28

wants to merge 4 commits into from

Conversation

hyprsyd
Copy link
Contributor

@hyprsyd hyprsyd commented Mar 23, 2023

No description provided.

@dol-sen dol-sen requested review from dol-sen and a team March 23, 2023 19:14
@dol-sen
Copy link
Contributor

dol-sen commented Mar 23, 2023

needs backup code to handle older portage versions without invalid_paths attribute as discussed in chat

@dol-sen
Copy link
Contributor

dol-sen commented Mar 23, 2023

Please also add a test for these changes

@hyprsyd
Copy link
Contributor Author

hyprsyd commented Mar 23, 2023

needs backup code to handle older portage versions without invalid_paths attribute as discussed in chat

On it

@thesamesam
Copy link
Member

  • Please try to add a test on both the Portage and gentoolkit side
  • Please add some detail to the commit message, including linking to the Portage PR, and the Gentoo bug.

@hyprsyd
Copy link
Contributor Author

hyprsyd commented Mar 24, 2023

  • Please try to add a test on both the Portage and gentoolkit side

    • Please add some detail to the commit message, including linking to the Portage PR, and the Gentoo bug.

On it

@hyprsyd hyprsyd requested a review from dol-sen April 6, 2023 13:19
@hyprsyd hyprsyd changed the title search : deal with invalid binpkgs eclean : deal with invalid binpkgs Apr 6, 2023
@hyprsyd
Copy link
Contributor Author

hyprsyd commented Apr 6, 2023

Screenshot-2023-04-06_01_35_44
Handling invalids
Screenshot-2023-04-06_01_08_37
Error if older portage is used

  changes required in portage -> gentoo/portage#1016
  gentoo bug -> https://bugs.gentoo.org/900224

Signed-off-by: Siddhanth Rathod <xsiddhanthrathod@gmail.com>
Modifying an incorrect error message
gentoo-bot pushed a commit to gentoo/portage that referenced this pull request Aug 21, 2023
Add object invalid_paths for new eclean feature to deal with invalid binpkgs.

This is needed for gentoolkit's eclean to handle cleaning up invalid binpkgs.

gentoolkit side PR: gentoo/gentoolkit#28
Bug: https://bugs.gentoo.org/900224
Signed-off-by: Siddhanth Rathod <xsiddhanthrathod@gmail.com>
Closes: #1016
Signed-off-by: Sam James <sam@gentoo.org>
@thesamesam
Copy link
Member

thesamesam commented Aug 21, 2023

Merged, thank you!

I'm sorry for taking ages and thank you for being so patient.

commit a16d0d4 (HEAD -> master, origin/master, origin/HEAD)
Author: Siddhanth Rathod xsiddhanthrathod@gmail.com
Date: Thu Mar 23 17:44:36 2023 +0530

eclean-pkg: deal with invalid binpkgs

Changes required in portage -> https://github.com/gentoo/portage/pull/1016

Bug: https://bugs.gentoo.org/900224
Signed-off-by: Siddhanth Rathod <xsiddhanthrathod@gmail.com>
Signed-off-by: Sam James <sam@gentoo.org>

commit d936026
Author: Siddhanth Rathod xsiddhanthrathod@gmail.com
Date: Thu Apr 6 20:47:28 2023 +0530

eclean: Add type hints

Signed-off-by: Siddhanth Rathod <xsiddhanthrathod@gmail.com>
Signed-off-by: Sam James <sam@gentoo.org>

@hyprsyd hyprsyd closed this by deleting the head repository Aug 22, 2023
palao pushed a commit to palao/portage that referenced this pull request Oct 16, 2023
Add object invalid_paths for new eclean feature to deal with invalid binpkgs.

This is needed for gentoolkit's eclean to handle cleaning up invalid binpkgs.

gentoolkit side PR: gentoo/gentoolkit#28
Bug: https://bugs.gentoo.org/900224
Signed-off-by: Siddhanth Rathod <xsiddhanthrathod@gmail.com>
Closes: gentoo#1016
Signed-off-by: Sam James <sam@gentoo.org>
palao pushed a commit to palao/portage that referenced this pull request Oct 22, 2023
Add object invalid_paths for new eclean feature to deal with invalid binpkgs.

This is needed for gentoolkit's eclean to handle cleaning up invalid binpkgs.

gentoolkit side PR: gentoo/gentoolkit#28
Bug: https://bugs.gentoo.org/900224
Signed-off-by: Siddhanth Rathod <xsiddhanthrathod@gmail.com>
Closes: gentoo#1016
Signed-off-by: Sam James <sam@gentoo.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants