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

Unindex packages when they are removed #10791

Closed
wants to merge 1 commit into from

Conversation

Chocimier
Copy link
Member

This prevents installing packages not delivered anymore by removing them from index rather than providing empty packages.

Removing current installations is covered by #9745.

Proposed workflow is to set removed=yes in template additionally to modifying template in way currently described in manual.
Template can be removed just after builders do the job, not months later.

If two phase process is not convenient enough, builders could unindex packages when templates are removed.

Requires void-linux/xbps#76.

@jnbr
Copy link
Contributor

jnbr commented Apr 11, 2019

Template can be removed just after builders do the job, not months later.

The problem is that it is not really known when a job is done. This would likely cause problems on independent builders such as ppc.

@Duncaen
Copy link
Member

Duncaen commented Apr 11, 2019

I commented on the other PR for the change in xbps-rindex about this: void-linux/xbps#76 (comment).

What I don't like about this proposed change in particular is, that it depends on a intermediate state of templates. This feels like something that is designed to just work with how current CI/build servers work.

I'd rather have xbps-checkvers output deleted packages and then have xbps-bulk or the future build server replacement handle this than xbps-src with pseudo bumps and a temporary keyword.

@Chocimier
Copy link
Member Author

Closing in favour of changes in xbps-bulk.

@Chocimier Chocimier closed this Jan 27, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants