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

Can we store link to commit which deleted a port? #108

Open
mojca opened this issue Sep 5, 2019 · 4 comments
Open

Can we store link to commit which deleted a port? #108

mojca opened this issue Sep 5, 2019 · 4 comments
Labels
enhancement New feature or request port info Provide more details info about ports priority:4-medium-low Medium low priority

Comments

@mojca
Copy link
Member

mojca commented Sep 5, 2019

When someone is looking for a particular (deleted) port, there might be a great chance that this person actually wants to install the port that everyone else thought was useless, or at least broken beyond repair. Maybe that person is even wiling to look into fixing the old Portfile.

What we are missing in MacPorts is precisely the way to easily find old deleted ports, and to find reasons why they were deleted. We have almost all functionality in place now to allow us to store the commit shasum in which the port was deleted, so we could provide URL to the commit on github, which would hopefully include the commit message explaining why the port was deleted, potentially also showing the link to the pull request with the relevant discussion. And more importantly, it would be easy to see what the contents of the old port were.

What do you think about adding either an additional column (or maybe an additional table since the number of entries is expected to be relatively low) listing (port_name, commit, <timestamp?>) for deleted ports?

As an example the port maven-ant-tasks could link to macports/macports-ports@493ec90.

Arjun already pointed out some potential issues with the approach I had in mind, but I wanted to open this as an issue to leave some room for brainstorming.

Originally posted by @mojca in https://github.com/macports/macports-webapp/pull/103/files

@arjunsalyan
Copy link
Member

Adding more comments related to deleted ports, I can create a new issue if this is the not the right place.

  1. Shall we show deleted ports in the search results by default?
  2. Shall we count them while counting ports under a category, maintainer, variant etc. ?

@pmetzger
Copy link
Member

pmetzger commented Sep 7, 2019

I'd suggest making it easy to find deleted ports, but not showing them by default. Maybe a checkbox.

@mojca
Copy link
Member Author

mojca commented Sep 8, 2019

I would say that the answer to both questions should be "NO".

While deleted ports still exist in the database, they should not in reality belong to any particular category, maintainer or variant (no need to remove that data from the database, but I would not show deleted ports when searching for ports belonging to a particular maintainer).

That said, we could have a completely separate search for deleted ports, but I would put that somewhere low on priority list. I don't see a good reason to show deleted ports anywhere, except, maybe, on the main page when searching for ports without any filter, but even then it would have to be at the end of the list, and clearly marked as deleted. Deleted ports might be nice for the information in case anyone is searching for a particular piece of software, serving as a hint that it was once present. But nothing more than that.

@mojca mojca added enhancement New feature or request port info Provide more details info about ports priority:4-medium-low Medium low priority labels Jul 21, 2021
@mascguy
Copy link
Member

mascguy commented Sep 1, 2022

While this sounds great in principle, tracking deleted ports indefinitely isn't ideal either. Has there been any thought/discussion, related to purging that info? My vote would be that we remove one year after deletion.

Though I'd be fine with not purging, if we no longer show them within various areas of the UI. Per issue: 283 - Deleted ports: don't show livecheck; change option "hide deleted" to "show deleted"

Thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request port info Provide more details info about ports priority:4-medium-low Medium low priority
Projects
None yet
Development

No branches or pull requests

4 participants