staged-recipes icon indicating copy to clipboard operation
staged-recipes copied to clipboard

Added recipe for Perl module Pod::Escapes

Open xileF1337 opened this issue 2 years ago • 4 comments

CondaForge port of recipe for Perl module Pod::Escapes. (It is currently on Bioconda only, but is a general-purpose module.)

Checklist

  • [x] Title of this PR is meaningful: e.g. "Adding my_nifty_package", not "updated meta.yaml".
  • [x] License file is packaged (see here for an example).
  • [x] Source is from official source.
  • [x] Package does not vendor other packages. (If a package uses the source of another package, they should be separate packages or the licenses of all packages need to be packaged).
  • [x] If static libraries are linked in, the license of the static library is packaged.
  • [x] Package does not ship static libraries. If static libraries are needed, follow CFEP-18.
  • [x] Build number is 0.
  • [x] A tarball (url) rather than a repo (e.g. git_url) is used in your recipe (see here for more details).
  • [x] GitHub users listed in the maintainer section have posted a comment confirming they are willing to be listed there.
  • [x] When in trouble, please check our knowledge base documentation before pinging a team.

xileF1337 avatar Sep 27 '22 13:09 xileF1337

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipes/perl-pod-escapes) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipes/perl-pod-escapes:

  • Recipe with the same name exists in bioconda: please discuss with @conda-forge/bioconda-recipes.

conda-forge-linter avatar Sep 27 '22 13:09 conda-forge-linter

@conda-forge/bioconda-recipes any objections against porting this to CondaForge?

xileF1337 avatar Sep 27 '22 13:09 xileF1337

I'm skipping the Windows build as current Perl seems to be unavailable:

- nothing provides perl >=5.32.1,<6.0a0 *_perl5 needed by perl-extutils-makemaker-7.36-pl5321hd8ed1ab_0
- nothing provides perl >=5.32.1,<6.0a0 *_perl5 needed by perl-exporter-5.72-pl5321hd8ed1ab_0

xileF1337 avatar Sep 27 '22 14:09 xileF1337

@conda-forge/help-perl, ready for review!

xileF1337 avatar Sep 27 '22 14:09 xileF1337

Hi friend!

We really, really, really appreciate that you have taken the time to make a PR on conda-forge/staged-recipes! conda-forge only exists because people like you donate their time to build and maintain conda recipes for use by the community.

In an effort to maintain this repository and increase the signal-to-noise for open PRs, the maintainers of staged-recipes close excessively old PRs after six months. This PR will remain open for another month, and then will be closed.

If you'd like to keep it open, please comment/push and we will be happy to oblige! Note that very old PRs will likely need to be rebased on main so that they can be rebuilt with the most recent CI scripts. If you have any trouble, or we missed reviewing this PR in the first place (sorry!), feel free to ping the team using a special command in a comment on the PR to get the attention of the staged-recipes team.

Cheers and thank you for contributing to this community effort!

stale[bot] avatar Apr 12 '23 02:04 stale[bot]

@xileF1337 You are still stuck here...? Same for me... No review on my recipe...

fgiacomoni avatar Apr 25 '23 06:04 fgiacomoni

@conda-forge-admin we need a review here

fgiacomoni avatar Apr 25 '23 06:04 fgiacomoni

@xileF1337 You are still stuck here...? Same for me... No review on my recipe...

Hey @fgiacomoni , well, I kind of abandoned this recipe because I realized that it is a Perl core module only after I opened the PR. Core modules are already included in the perl package and do not need to be specified as separate requirements.

Concerning your own recipe, feel free to mention @conda-forge/perl-packagers to get a review. Only few people from the conda-forge team are using Perl, so it can take a moment. Currently, we are only two people in the perl-packagers team.

I'm closing this PR since the recipe should probably not be merged.

xileF1337 avatar Apr 29 '23 11:04 xileF1337

I take it you refer to https://github.com/conda-forge/staged-recipes/pull/22648 .

xileF1337 avatar May 03 '23 19:05 xileF1337