[ITP] perl-Parse-Yapp-1.21
This is a straightforward port, akin to other Perl packages. It is already available in numerous Linux distros (and FreeBSD): https://pkgs.org/download/perl(Parse::Yapp) License: "Artistic-1.0" OR "GPL-2.0-or-later". Running 'cygport test' yields no failures. https://drive.google.com/file/d/1gYVsNOOj0-YJEB2uWEY4NjArklzcyUmT/view?usp=sharing https://drive.google.com/file/d/1H1Kf8IeZQtadXBCg7QDZRuv8_hPSABNv/view?usp=sharing https://drive.google.com/file/d/1SFYn8aeHfeLXr4rguj05QVxKCYe4s72B/view?usp=sharing https://drive.google.com/file/d/1OzEATt9Y7rarnEa-nDJ3Cks01FLvL1NJ/view?usp=sharing I can be the maintainer-at-large for this if you'd like. --Zem
Re: [ITA] bash-completion/-devel
On 2024-05-03 07:40, Jon Turney via Cygwin-apps wrote:
On 29/04/2024 22:13, Brian Inglis via Cygwin-apps wrote:
I would like to co-maintain or adopt and revive the above package, which was
adopted by Eric but not updated since Yaakov.
Thanks.
I added this to your packages.
Thanks Jon
I guess I need to ask eblake if he wants to orphan his packages, since he seems
to be no longer active...
Looks like he's busy with Austin Group POSIX + IEEE update plus work
Below are links to existing source packages, build repos, scallywag runs, and
updated package info.
I would like to further improve the sdesc and ldesc provided to reflect that
completions are provided for thousands of commands and their options and
arguments.
Bash Completions and development
Existing source package:
https://cygwin.com/packages/summary/bash-completion-src.html
Updated cygport:
https://cygwin.com/cgit/cygwin-packages/bash-completion/tree/bash-completion.cygport?h=playground
A few comments:
DEPEND="automake dejagnu make screen" # tcllib
BUILD_REQUIRES="$DEPEND"
Just set BUILD_REQUIRES.
I assume the comment about tcllib means something to someone. :)
Checked Fedora spec for "advice" - required for testing - n/a on Cygwin
src_test() {
cd $B
# For some tests involving non-ASCII filenames
export LANG=C.UTF-8
export -f cygtest
# This stuff borrowed from dejagnu-1.4.4-17 (tests need a terminal)
tmpfile=$(mktemp bash-completion.screen.XX.tmp)
# cygtest
At first glance, because this is commented out, I thought "so this doesn't run
tests at all"
Maybe remove the commented out line, and write a comment saying something like
"run tests under screen, since they need a terminal"
screen -D -m bash -c '( cygtest ; echo $? ) >'$tmpfile
cat $tmpfile
result=$(tail -n 1 $tmpfile)
Also borrowed from Fedora spec.
Plan to do more comments and cleanup before merging to master.
They also had a ChangeLog generation problem so reported upstream and made my
own; they are trying a fix and update or may have a newer package release.
This cleverness to propagate the exit code is probably also worthy of comment,
since I had to think about it for a few minutes before I realized what it was
doing...
Perhaps should remove tmpfile here?
Must have deleted instead of commenting out!
return $result
}
--
Take care. Thanks, Brian Inglis Calgary, Alberta, Canada
La perfection est atteinte Perfection is achieved
non pas lorsqu'il n'y a plus rien à ajouter not when there is no more to add
mais lorsqu'il n'y a plus rien à retirer but when there is no more to cut
-- Antoine de Saint-Exupéry
Re: [ITA] bash-completion/-devel
On 29/04/2024 22:13, Brian Inglis via Cygwin-apps wrote:
I would like to co-maintain or adopt and revive the above package, which
was adopted by Eric but not updated since Yaakov.
Thanks.
I added this to your packages.
I guess I need to ask eblake if he wants to orphan his packages, since
he seems to be no longer active...
Below are links to existing source packages, build repos, scallywag
runs, and updated package info.
I would like to further improve the sdesc and ldesc provided to reflect
that completions are provided for thousands of commands and their
options and arguments.
Bash Completions and development
Existing source package:
https://cygwin.com/packages/summary/bash-completion-src.html
Updated cygport:
https://cygwin.com/cgit/cygwin-packages/bash-completion/tree/bash-completion.cygport?h=playground
A few comments:
DEPEND="automake dejagnu make screen" # tcllib
BUILD_REQUIRES="$DEPEND"
Just set BUILD_REQUIRES.
I assume the comment about tcllib means something to someone. :)
src_test() {
cd $B
# For some tests involving non-ASCII filenames
export LANG=C.UTF-8
export -f cygtest
# This stuff borrowed from dejagnu-1.4.4-17 (tests need a terminal)
tmpfile=$(mktemp bash-completion.screen.XX.tmp)
# cygtest
At first glance, because this is commented out, I thought "so this
doesn't run tests at all"
Maybe remove the commented out line, and write a comment saying
something like "run tests under screen, since they need a terminal"
screen -D -m bash -c '( cygtest ; echo $? ) >'$tmpfile
cat $tmpfile
> result=$(tail -n 1 $tmpfile)
This cleverness to propagate the exit code is probably also worthy of
comment, since I had to think about it for a few minutes before I
realized what it was doing...
Perhaps should remove tmpfile here?
return $result
}
