[ITP] cbonsai 1.0.4

2021-05-26 Thread Daisuke Fujimura via Cygwin-apps
Hello,

[ITP] A new package proposal: cbonsai

- cbonsai



SUMMARY: Grow bonsai trees in your terminal
HOMEPAGE: https://gitlab.com/jallbrit/cbonsai
SRC_URL: 
https://gitlab.com/jallbrit/cbonsai/-/archive/v1.0.4/cbonsai-v1.0.4.tar.bz2
LICENSE: GPLv3



Corresponding Linux/Unix packages are searched:
- https://repology.org/project/cbonsai/versions

Cygport
- https://yacp.osdn.jp/itp/cbonsai/cbonsai.cygport

CI (playground):
- https://cygwin.com/cgi-bin2/jobs.cgi?id=2880
- https://ci.appveyor.com/project/cygwin/scallywag/builds/39341570



Since this programs require keystrokes, I tested it manually, as
shown below.

```
$ cd ${B}
$ ./${PN}.exe
```


Re: [ITP] tty-clock 2.3

2021-05-26 Thread Daisuke Fujimura via Cygwin-apps
Thank you for your approval.

On Thu, May 27, 2021 at 1:03 AM Marco Atzeri via Cygwin-apps
 wrote:
>
>
> On 25.05.2021 23:52, Daisuke Fujimura via Cygwin-apps wrote:
> > Sorry, typo in the link to the review target.
> >
> >> Cygport, packages and logs:
> >> - https://yacp.osdn.jp/itp/tty-clocks/
> >
> > https://yacp.osdn.jp/itp/tty-clock/
> >
> >
> >> - added BUILD_REQUIRES
> >> - defined VERSION="2.3" and RELEASE=1 explicitly
> >
> > https://yacp.osdn.jp/itp/tty-clock/tty-clock.cygport already solves
> > the above problem.
> >
> > Can you please review it (
> > https://yacp.osdn.jp/itp/tty-clock/tty-clock.cygport ) again?
> >
>
> I added tty-clock to the list of packages, you can upload
>
> Regards
> Marco
>


Re: [PATCH] cygutils/cygdrop: fix return type of usageCore

2021-05-26 Thread Mark Geisert

Hi Jeremy,

Jeremy Drake via Cygwin-apps wrote:

Fixes a warning "no return statement in function returning non-void",
and solves a crash running --help.

Hopefully this is the right place for this now, since I am not interesting
in becoming a package maintainer as the list description says ;)


Thanks for the report and patch.  I can't reproduce the crash but no worries.  The 
patch will be part of the next update to cygutils in a short while.


..mark


[PATCH cygport] Add 'vars' command to output arbitrary .cygport variables

2021-05-26 Thread Jon Turney
Since variables in a .cygport file can be computed from arbitrary shell
expressions, perhaps containing variables that cygport itself defines,
other programs can't determine their value just by parsing the .cygport
file.

Add 'vars' command to output arbitrary variables defined by the .cygport
file.

The idea is that the output from this should be eval-able in a shell to
be further used, but it probably needs some additional escaping (of
newlines?) to be completely right in all cases.

e.g

> $ cygport libX11.cygport vars PVR BUILD_REQUIRES
> PVR='1.7.1-1'
> BUILD_REQUIRES='xtrans xorgproto libxcb-devel xmlto xorg-sgml-doctools 
> docbook-xml43'

> eval $(cygport libX11.cygport vars BUILD_REQUIRES) ; for r in 
> ${BUILD_REQUIRES} ; do echo $r; done
> xtrans
> xorgproto
> libxcb-devel
> xmlto
> xorg-sgml-doctools
> docbook-xml43
---

Notes:
This is a generalization of the idea in [1], for querying the
build-depends of a package.

https://cygwin.com/pipermail/cygwin-apps/2021-January/041006.html

 bin/cygport.in   |  5 +
 data/cygport-bash-completion |  2 +-
 lib/pkg_info.cygpart | 10 +-
 3 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/bin/cygport.in b/bin/cygport.in
index afc6f7e..e9295be 100755
--- a/bin/cygport.in
+++ b/bin/cygport.in
@@ -623,6 +623,11 @@ do
__show_info;
_status=$?;
;;
+   vars)
+   __show_vars ${argv[@]:$((++arg_n))};
+   _status=$?;
+   arg_n=$argc;  # consumed all remaining args
+   ;;
homepage|web*|www*)
__show_web;
_status=$?;
diff --git a/data/cygport-bash-completion b/data/cygport-bash-completion
index 86df238..9085e19 100644
--- a/data/cygport-bash-completion
+++ b/data/cygport-bash-completion
@@ -12,7 +12,7 @@ _cygport()
check test inst install list dep depends \
package pkg package-test pkg-test diff mkdiff mkpatch \
upload stage announce almostall all all-test clean finish \
-   help info version homepage website'
+   help info version homepage website vars'
 
if [[ $COMP_CWORD -eq 1 ]] ; then
# first arg: source file completion
diff --git a/lib/pkg_info.cygpart b/lib/pkg_info.cygpart
index 4b18993..a5475cc 100644
--- a/lib/pkg_info.cygpart
+++ b/lib/pkg_info.cygpart
@@ -658,5 +658,13 @@ __show_web() {
${mybrowser} ${mywww} &>/dev/null &
 }
 
+__show_vars() {
+   for v in "$@"
+   do
+   echo "${v}='${!v}'"
+   done
+}
+
 readonly -f __list_files __list_debug_files __list_deps \
-   __show_deps __show_info __show_web __pager
+   __show_deps __show_info __show_web __pager \
+   __show_vars
-- 
2.31.1



[PATCH] cygutils/cygdrop: fix return type of usageCore

2021-05-26 Thread Jeremy Drake via Cygwin-apps
Fixes a warning "no return statement in function returning non-void",
and solves a crash running --help.

Hopefully this is the right place for this now, since I am not interesting
in becoming a package maintainer as the list description says ;)From f4821db24d4e4feca16e4aea58843128e233ef4e Mon Sep 17 00:00:00 2001
From: Jeremy Drake 
Date: Tue, 25 May 2021 16:13:17 -0700
Subject: [PATCH] cygdrop: fix return type of usageCore

Fixes a warning "no return statement in function returning non-void",
and solves a crash running --help.
---
 src/cygdrop/cygdrop.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/cygdrop/cygdrop.cc b/src/cygdrop/cygdrop.cc
index 35bcc19..dc403c9 100644
--- a/src/cygdrop/cygdrop.cc
+++ b/src/cygdrop/cygdrop.cc
@@ -39,7 +39,7 @@ static void help (FILE * f, const char *name);
 static void version (FILE * f, const char *name);
 static void license (FILE * f, const char *name);
 
-static int
+static void
 usageCore (FILE * f, const char * name)
 {
   fprintf (f,
-- 
2.31.1



Re: [ITP] tty-clock 2.3

2021-05-26 Thread Marco Atzeri via Cygwin-apps



On 25.05.2021 23:52, Daisuke Fujimura via Cygwin-apps wrote:

Sorry, typo in the link to the review target.


Cygport, packages and logs:
- https://yacp.osdn.jp/itp/tty-clocks/


https://yacp.osdn.jp/itp/tty-clock/



- added BUILD_REQUIRES
- defined VERSION="2.3" and RELEASE=1 explicitly


https://yacp.osdn.jp/itp/tty-clock/tty-clock.cygport already solves
the above problem.

Can you please review it (
https://yacp.osdn.jp/itp/tty-clock/tty-clock.cygport ) again?



I added tty-clock to the list of packages, you can upload

Regards
Marco