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

glyr: fix test, update license #70505

Merged
merged 1 commit into from
Feb 5, 2021
Merged

Conversation

carlocab
Copy link
Member

@carlocab carlocab commented Feb 5, 2021

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

This fixes CI failures found in #66450, #70319, #70419, etc.

Also, update the license. From src/glyrc/glryc.c:

/***********************************************************
 * This file is part of glyr
 * + a commnadline tool and library to download various sort of music related metadata.
 * + Copyright (C) [2011]  [Christopher Pahl]
 * + Hosted at: https://github.com/sahib/glyr
 *
 * glyr is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Lesser General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * glyr is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU Lesser General Public License for more details.
 *
 * You should have received a copy of the GNU Lesser General Public License
 * along with glyr. If not, see <http://www.gnu.org/licenses/>.
 **************************************************************/

This fixes CI failures found in Homebrew#66450, Homebrew#70319, Homebrew#70419, etc.

Also, update the license. From src/glyrc/glryc.c:

    /***********************************************************
     * This file is part of glyr
     * + a commnadline tool and library to download various sort of music related metadata.
     * + Copyright (C) [2011]  [Christopher Pahl]
     * + Hosted at: https://github.com/sahib/glyr
     *
     * glyr is free software: you can redistribute it and/or modify
     * it under the terms of the GNU Lesser General Public License as published by
     * the Free Software Foundation, either version 3 of the License, or
     * (at your option) any later version.
     *
     * glyr is distributed in the hope that it will be useful,
     * but WITHOUT ANY WARRANTY; without even the implied warranty of
     * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
     * GNU Lesser General Public License for more details.
     *
     * You should have received a copy of the GNU Lesser General Public License
     * along with glyr. If not, see <http://www.gnu.org/licenses/>.
     **************************************************************/
@carlocab
Copy link
Member Author

carlocab commented Feb 5, 2021

We don't need newer bottles, and there's no reason to make the existing older bottles inaccessible. Merging.

@carlocab carlocab merged commit e43a7bd into Homebrew:master Feb 5, 2021
@carlocab carlocab deleted the glyr-test branch February 5, 2021 14:24
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Mar 8, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Mar 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
license License PRs outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants