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

py-graveyard: purge entries that are more than one year old #5383

Merged
merged 1 commit into from Sep 29, 2019

Conversation

reneeotten
Copy link
Contributor

Description

Remove entries in that have been present for more than one year, compared to py-graveyard/Portfile at commit 754246c (30Sep2018).

We typically purge obsoleted ports after one year, so there is no reason to leave the replacement rules in place for longer than this. This should significantly speed up the parsing of the file.

See: https://trac.macports.org/ticket/59051

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.14.6 18G95
Xcode 10.3 10G8

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL?
  • checked your Portfile with port lint? Reported warnings/errors do not apply to the py-graveyard port
  • tried existing tests with sudo port test? N/A
  • tried a full install with sudo port -vst install? N/A
  • tested basic functionality of all binary files? N/A

Remove entries in that have been present for more than one year,
compared to py-graveyard/Portfile at commit 754246c (30Sep2018).

We typically purge obsoleted ports after one year, so there is no reason
to leave the replacement rules in place for longer than this.

See: https://trac.macports.org/ticket/59051
@macportsbot macportsbot added by: member Created by a member with commit rights maintainer: none type: enhancement labels Sep 27, 2019
@macportsbot
Copy link

Travis Build #8430 Passed.

Lint results
--->  Verifying Portfile for py-graveyard
Error: Missing required variable: platforms
Error: Missing required variable: homepage
Warning: no license set
--->  2 errors and 1 warnings found.

@reneeotten
Copy link
Contributor Author

@mojca @mf2k @pmetzger any concerns about this?

@mf2k
Copy link
Contributor

mf2k commented Sep 28, 2019

No. LGTM.

@reneeotten reneeotten merged commit 006a4c2 into macports:master Sep 29, 2019
@reneeotten reneeotten deleted the py-graveyard branch September 29, 2019 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
by: member Created by a member with commit rights maintainer: none type: enhancement
4 participants