Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] New package: xmenu-4.4.1
@ 2021-01-08 16:00 kawaiiamber
  2021-01-08 16:02 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (79 more replies)
  0 siblings, 80 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 16:00 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 897 bytes --]

There is a new pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I uncluded that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1018 bytes --]

From fb8de5863ec1487757e510675112a37e17fc0568 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..ed9204d83df
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,16 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="x11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
@ 2021-01-08 16:02 ` kawaiiamber
  2021-01-08 17:06 ` ericonr
                   ` (78 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 16:02 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 902 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I uncluded that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1018 bytes --]

From 63ec96528fa06ce5019b12dc6128a30059a2e9ee Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..288366c1a83
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,16 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
  2021-01-08 16:02 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 17:06 ` ericonr
  2021-01-08 17:06 ` ericonr
                   ` (77 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ericonr @ 2021-01-08 17:06 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 197 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756879987

Comment:
Maybe our `Xft.h` header include `ft2build.h` somewhere.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
  2021-01-08 16:02 ` [PR PATCH] [Updated] " kawaiiamber
  2021-01-08 17:06 ` ericonr
@ 2021-01-08 17:06 ` ericonr
  2021-01-08 17:07 ` ericonr
                   ` (76 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ericonr @ 2021-01-08 17:06 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 198 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756879987

Comment:
Maybe our `Xft.h` header includes `ft2build.h` somewhere.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (2 preceding siblings ...)
  2021-01-08 17:06 ` ericonr
@ 2021-01-08 17:07 ` ericonr
  2021-01-08 17:16 ` kawaiiamber
                   ` (75 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ericonr @ 2021-01-08 17:07 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 295 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756879987

Comment:
Maybe our `Xft.h` header includes `ft2build.h` somewhere. It could be missing the `-I${XBPS_CROSS_BASE}/usr/include/freetype2` option in `CFLAGS` as well.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (3 preceding siblings ...)
  2021-01-08 17:07 ` ericonr
@ 2021-01-08 17:16 ` kawaiiamber
  2021-01-08 17:19 ` kawaiiamber
                   ` (74 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 17:16 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 311 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756888891

Comment:
Interesting. I'm still learning xbps-src. If I'm not mistaken, would this be the right syntax?
```
make_build_args="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
```

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (4 preceding siblings ...)
  2021-01-08 17:16 ` kawaiiamber
@ 2021-01-08 17:19 ` kawaiiamber
  2021-01-08 17:20 ` kawaiiamber
                   ` (73 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 17:19 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 379 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756890475

Comment:
Oh, also, it does seem that VOID includes `ft2buiild.h` in their `Xft.h` header.
```sh
[amber@otaku ~]$ xlocate Xft.h
libXft-devel-2.3.3_1    /usr/include/X11/Xft/Xft.h
```
On line 39 of `Xft.h`:
```c
#include <ft2build.h>
```

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (5 preceding siblings ...)
  2021-01-08 17:19 ` kawaiiamber
@ 2021-01-08 17:20 ` kawaiiamber
  2021-01-08 17:36 ` kawaiiamber
                   ` (72 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 17:20 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 400 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756890475

Comment:
Oh, also, it does seem that VOID includes `ft2buiild.h` in their `Xft.h` header.
```sh
[amber@otaku ~]$ xlocate Xft.h
libXft-devel-2.3.3_1    /usr/include/X11/Xft/Xft.h
```
On line 39 of `/usr/include/X11/Xft/Xft.h`:
```c
#include <ft2build.h>
```

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (6 preceding siblings ...)
  2021-01-08 17:20 ` kawaiiamber
@ 2021-01-08 17:36 ` kawaiiamber
  2021-01-08 17:41 ` kawaiiamber
                   ` (71 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 17:36 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 995 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756898562

Comment:
> Maybe our `Xft.h` header includes `ft2build.h` somewhere. It could be missing the `-I${XBPS_CROSS_BASE}/usr/include/freetype2` option in `CFLAGS` as well.

From the upstream `config.mk` which is included in the `Makefile`:
```
# includes and libs
INCS := -I${LOCALINC} -I${X11INC} -I${FREETYPEINC}
LIBS := -L${LOCALLIB} -L${X11LIB} -lfontconfig -lXft -lX11 -lXinerama -lImlib2

# flags
CPPFLAGS :=
CFLAGS   := -Wall -Wextra ${INCS} ${CPPFLAGS}
LDFLAGS  := ${LIBS}
```
This results in `make` doing:
```
[amber@otaku ~/.local/src/xmenu]$ make
cc -Wall -Wextra -I/usr/local/include -I/usr/X11R6/include -I/usr/include/freetype2  -c xmenu.c
cc -o xmenu xmenu.o -L/usr/local/lib -L/usr/X11R6/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2
```
Thus, I don't think `-I${XBPS_CROSS_BASE}/usr/include/freetype2` is missing from `CFLAGS`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (7 preceding siblings ...)
  2021-01-08 17:36 ` kawaiiamber
@ 2021-01-08 17:41 ` kawaiiamber
  2021-01-08 17:47 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (70 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 17:41 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1285 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756898562

Comment:
> Maybe our `Xft.h` header includes `ft2build.h` somewhere. It could be missing the `-I${XBPS_CROSS_BASE}/usr/include/freetype2` option in `CFLAGS` as well.

From the upstream `config.mk` which is included in the `Makefile`:
```
# paths
PREFIX    ?= /usr/local
MANPREFIX ?= ${PREFIX}/share/man

LOCALINC ?= /usr/local/include
LOCALLIB ?= /usr/local/lib

X11INC ?= /usr/X11R6/include
X11LIB ?= /usr/X11R6/lib

FREETYPEINC ?= /usr/include/freetype2
# OpenBSD (uncomment)
#FREETYPEINC = ${X11INC}/freetype2

# includes and libs
INCS := -I${LOCALINC} -I${X11INC} -I${FREETYPEINC}
LIBS := -L${LOCALLIB} -L${X11LIB} -lfontconfig -lXft -lX11 -lXinerama -lImlib2

# flags
CPPFLAGS :=
CFLAGS   := -Wall -Wextra ${INCS} ${CPPFLAGS}
LDFLAGS  := ${LIBS}
```
This results in `make` doing:
```
[amber@otaku ~/.local/src/xmenu]$ make
cc -Wall -Wextra -I/usr/local/include -I/usr/X11R6/include -I/usr/include/freetype2  -c xmenu.c
cc -o xmenu xmenu.o -L/usr/local/lib -L/usr/X11R6/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2
```
Thus, I don't think `-I${XBPS_CROSS_BASE}/usr/include/freetype2` is missing from `CFLAGS`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (8 preceding siblings ...)
  2021-01-08 17:41 ` kawaiiamber
@ 2021-01-08 17:47 ` kawaiiamber
  2021-01-08 17:55 ` kawaiiamber
                   ` (69 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 17:47 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1033 bytes --]

From 5006a46d0ece522df00c24b10c0c1e0610ecf090 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..7811f9f6769
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,16 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel freetype-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (9 preceding siblings ...)
  2021-01-08 17:47 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 17:55 ` kawaiiamber
  2021-01-08 17:59 ` kawaiiamber
                   ` (68 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 17:55 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1043 bytes --]

From b5480d894f065ac3e275c81242164d1481715a64 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..af429ae8966
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,16 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel xorgproto freetype-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (10 preceding siblings ...)
  2021-01-08 17:55 ` kawaiiamber
@ 2021-01-08 17:59 ` kawaiiamber
  2021-01-08 18:11 ` kawaiiamber
                   ` (67 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 17:59 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1028 bytes --]

From 128e8fbcd9d995e237997ebf8a707dc456fbc206 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..bb4071e4066
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,16 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel xorgproto"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (11 preceding siblings ...)
  2021-01-08 17:59 ` kawaiiamber
@ 2021-01-08 18:11 ` kawaiiamber
  2021-01-08 18:14 ` kawaiiamber
                   ` (66 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 18:11 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1164 bytes --]

From bbc4b6b0039fe057ce05f9fc9d989db0befb743a Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..0a48af99cea
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,25 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+# gnu-makefile build-style complains about missing ft2build.h
+# build_style=gnu-makefile
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel xorgproto"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+do_build() {
+	make
+}
+
+do_install() {
+	make install
+}
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (12 preceding siblings ...)
  2021-01-08 18:11 ` kawaiiamber
@ 2021-01-08 18:14 ` kawaiiamber
  2021-01-08 18:22 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (65 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 18:14 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 345 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756916855

Comment:
Doing `build-style=gnu-makefile` fails with complaints about an unused `ft2build.h` header file. For some reason, specifying `make` and `make install` in `do_build()` and `do_install()` seems to work.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (13 preceding siblings ...)
  2021-01-08 18:14 ` kawaiiamber
@ 2021-01-08 18:22 ` kawaiiamber
  2021-01-08 18:26 ` kawaiiamber
                   ` (64 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 18:22 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1154 bytes --]

From 63da1a3821fc1dd9e66b8768e1daefdaedf09325 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..0f33ac97032
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,25 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+# gnu-makefile build-style complains about missing ft2build.h
+# build_style=gnu-makefile
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+do_build() {
+	make
+}
+
+do_install() {
+	make install
+}
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (14 preceding siblings ...)
  2021-01-08 18:22 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 18:26 ` kawaiiamber
  2021-01-08 18:31 ` kawaiiamber
                   ` (63 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 18:26 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1154 bytes --]

From 920a8144aac303438e97c64fcba4741e379ae2a3 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..86448cdcb27
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,25 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+# gnu-makefile build_style complains about missing ft2build.h
+# build_style=gnu-makefile
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+do_build() {
+	make
+}
+
+do_install() {
+	make install
+}
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (15 preceding siblings ...)
  2021-01-08 18:26 ` kawaiiamber
@ 2021-01-08 18:31 ` kawaiiamber
  2021-01-08 18:34 ` kawaiiamber
                   ` (62 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 18:31 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 463 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756925114

Comment:
The `xmenu.c` file includes the `Xatom` header file.
```c
#include <X11/Xatom.h>
```
Using `xlocate`, it seems that it's from `xorgproto`
```sh
[amber@otaku ~]$ xlocate Xatom.h
xorgproto-2020.1_1      /usr/include/X11/Xatom.h
```
However, it seems to be okay even when I removed xorgproto from `hostmakedeps`.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (16 preceding siblings ...)
  2021-01-08 18:31 ` kawaiiamber
@ 2021-01-08 18:34 ` kawaiiamber
  2021-01-08 18:36 ` kawaiiamber
                   ` (61 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 18:34 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 549 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756925114

Comment:
The `xmenu.c` file includes the `Xatom` header file.
```c
#include <X11/Xatom.h>
```
Using `xlocate`, it seems that it's from `xorgproto`
```sh
[amber@otaku ~]$ xlocate Xatom.h
xorgproto-2020.1_1      /usr/include/X11/Xatom.h
```
However, it seems to be okay even when I removed xorgproto from `hostmakedeps`.

# EDIT
Nevermind, `xorgproto` is installed with `libX11-devel` and `libXft-devel`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (17 preceding siblings ...)
  2021-01-08 18:34 ` kawaiiamber
@ 2021-01-08 18:36 ` kawaiiamber
  2021-01-08 20:32 ` [PR REVIEW] " ndowens
                   ` (60 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 18:36 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 549 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756925114

Comment:
The `xmenu.c` file includes the `Xatom` header file.
```c
#include <X11/Xatom.h>
```
Using `xlocate`, it seems that it's from `xorgproto`
```sh
[amber@otaku ~]$ xlocate Xatom.h
xorgproto-2020.1_1      /usr/include/X11/Xatom.h
```
However, it seems to be okay even when I removed xorgproto from `hostmakedeps`.

# EDIT
Nevermind, `xorgproto` is installed with `libX11-devel` and `libXft-devel`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (18 preceding siblings ...)
  2021-01-08 18:36 ` kawaiiamber
@ 2021-01-08 20:32 ` ndowens
  2021-01-08 20:38 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (59 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 20:32 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 178 bytes --]

New review comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r554176378

Comment:
Remove the commented out lines 

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (19 preceding siblings ...)
  2021-01-08 20:32 ` [PR REVIEW] " ndowens
@ 2021-01-08 20:38 ` kawaiiamber
  2021-01-08 20:38 ` [PR REVIEW] " kawaiiamber
                   ` (58 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:38 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1061 bytes --]

From b417b09086109623d9fe42a48b039be318f3120e Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..c6e16533a9c
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,23 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+hostmakedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+do_build() {
+	make
+}
+
+do_install() {
+	make install
+}
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (20 preceding siblings ...)
  2021-01-08 20:38 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 20:38 ` kawaiiamber
  2021-01-08 20:43 ` ndowens
                   ` (57 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:38 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 155 bytes --]

New review comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r554179173

Comment:
Done

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (21 preceding siblings ...)
  2021-01-08 20:38 ` [PR REVIEW] " kawaiiamber
@ 2021-01-08 20:43 ` ndowens
  2021-01-08 20:44 ` ndowens
                   ` (56 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 20:43 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 205 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756989319

Comment:
Just looked at CI and files in package and it only shows license

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (22 preceding siblings ...)
  2021-01-08 20:43 ` ndowens
@ 2021-01-08 20:44 ` ndowens
  2021-01-08 20:50 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (55 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 20:44 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 247 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756989727

Comment:
And reason I think you had issue is because you have -devel in hostmakedepends which should be makedepends

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (23 preceding siblings ...)
  2021-01-08 20:44 ` ndowens
@ 2021-01-08 20:50 ` kawaiiamber
  2021-01-08 20:50 ` kawaiiamber
                   ` (54 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:50 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1057 bytes --]

From f86c9e6ad01c171385e0cc276e58a4c796641de1 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..c55bee3144e
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,23 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+do_build() {
+	make
+}
+
+do_install() {
+	make install
+}
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (24 preceding siblings ...)
  2021-01-08 20:50 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 20:50 ` kawaiiamber
  2021-01-08 20:51 ` ndowens
                   ` (53 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:50 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 300 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756992058

Comment:
> And reason I think you had issue is because you have -devel in hostmakedepends which should be makedepends

Moved stuff in `hostmakedeps` to `makedeps`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (25 preceding siblings ...)
  2021-01-08 20:50 ` kawaiiamber
@ 2021-01-08 20:51 ` ndowens
  2021-01-08 20:52 ` kawaiiamber
                   ` (52 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 20:51 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 195 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756992172

Comment:
Working on a fix for build. As it will error for cross

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (26 preceding siblings ...)
  2021-01-08 20:51 ` ndowens
@ 2021-01-08 20:52 ` kawaiiamber
  2021-01-08 20:53 ` ndowens
                   ` (51 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:52 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 243 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756992782

Comment:
After changing `hostmakedeps` to `makedeps`, `build_style=gnu-makefile` still produces same error.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (27 preceding siblings ...)
  2021-01-08 20:52 ` kawaiiamber
@ 2021-01-08 20:53 ` ndowens
  2021-01-08 20:53 ` kawaiiamber
                   ` (50 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 20:53 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 268 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756992893

Comment:
> After changing `hostmakedeps` to `makedeps`, `build_style=gnu-makefile` still produces same error.

Yup, I am working on it

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (28 preceding siblings ...)
  2021-01-08 20:53 ` ndowens
@ 2021-01-08 20:53 ` kawaiiamber
  2021-01-08 20:54 ` kawaiiamber
                   ` (49 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:53 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 299 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756992782

Comment:
After changing `hostmakedeps` to `makedeps`, `build_style=gnu-makefile` still produces same error. But now it seems to be failing more checks than before.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (29 preceding siblings ...)
  2021-01-08 20:53 ` kawaiiamber
@ 2021-01-08 20:54 ` kawaiiamber
  2021-01-08 20:54 ` kawaiiamber
                   ` (48 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:54 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 369 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756992782

Comment:
After changing `hostmakedeps` to `makedeps`, `build_style=gnu-makefile` still produces same error. But now it seems to be failing more checks than before. It'd odd as `./xbps-src pkg xmenu` seems to pass locally on my end...

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (30 preceding siblings ...)
  2021-01-08 20:54 ` kawaiiamber
@ 2021-01-08 20:54 ` kawaiiamber
  2021-01-08 20:56 ` kawaiiamber
                   ` (47 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:54 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 369 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756992782

Comment:
After changing `hostmakedeps` to `makedeps`, `build_style=gnu-makefile` still produces same error. But now it seems to be failing more checks than before. It's odd as `./xbps-src pkg xmenu` seems to pass locally on my end...

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (31 preceding siblings ...)
  2021-01-08 20:54 ` kawaiiamber
@ 2021-01-08 20:56 ` kawaiiamber
  2021-01-08 21:51 ` kawaiiamber
                   ` (46 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 20:56 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 749 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-756994221

Comment:
On the archs that the check fails, it reports that
```sh
cc -Wall -Wextra -I/usr/local/include -I/usr/X11R6/include -I/usr/include/freetype2  -c xmenu.c
xmenu.c:10:10: fatal error: X11/Xlib.h: No such file or directory
   10 | #include <X11/Xlib.h>
      |          ^~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:14: xmenu.o] Error 1
=> ERROR: xmenu-4.4.1_1: do_build: 'make' exited with 2
=> ERROR:   in do_build() at srcpkgs/xmenu/template:14
Error: Process completed with exit code 1.
```
I wonder why some archs are failing to find the `Xlib.h` header file while others succeed.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (32 preceding siblings ...)
  2021-01-08 20:56 ` kawaiiamber
@ 2021-01-08 21:51 ` kawaiiamber
  2021-01-08 22:00 ` ndowens
                   ` (45 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 21:51 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 217 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757014882

Comment:
I wonder if it's worth it to add `archs="x86_64* i686"` to the template.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (33 preceding siblings ...)
  2021-01-08 21:51 ` kawaiiamber
@ 2021-01-08 22:00 ` ndowens
  2021-01-08 22:00 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (44 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 22:00 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 280 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757020328

Comment:
> I wonder if it's worth it to add `archs="x86_64* i686"` to the template.

No. Try this https://termbin.com/qmqq

Seems to work for me

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (34 preceding siblings ...)
  2021-01-08 22:00 ` ndowens
@ 2021-01-08 22:00 ` kawaiiamber
  2021-01-08 22:03 ` ndowens
                   ` (43 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 22:00 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1080 bytes --]

From 7e64b28131e5458492697b401ab67e5718053da1 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..3444a210259
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,24 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+archs="x86_64* i686"
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+do_build() {
+	make
+}
+
+do_install() {
+	make install
+}
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (35 preceding siblings ...)
  2021-01-08 22:00 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 22:03 ` ndowens
  2021-01-08 22:03 ` ndowens
                   ` (42 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 22:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 343 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757020328

Comment:
> I wonder if it's worth it to add `archs="x86_64* i686"` to the template.

No. Try this https://termbin.com/qmqq

Seems to work for me

Change -L${XBPS_CROSS_BASE} to -L${XBPS_CROSS_BASE}/usr/lib

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (36 preceding siblings ...)
  2021-01-08 22:03 ` ndowens
@ 2021-01-08 22:03 ` ndowens
  2021-01-08 22:03 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (41 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 22:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 392 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757020328

Comment:
> I wonder if it's worth it to add `archs="x86_64* i686"` to the template.

No. Try this https://termbin.com/qmqq

Seems to work for me

Change -L${XBPS_CROSS_BASE} to -L${XBPS_CROSS_BASE}/usr/lib 
in that paste, forgot to add the /usr/lib part

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (37 preceding siblings ...)
  2021-01-08 22:03 ` ndowens
@ 2021-01-08 22:03 ` kawaiiamber
  2021-01-08 22:05 ` kawaiiamber
                   ` (40 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 22:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1172 bytes --]

From 255c81a06dc7f431dd56790c441634afe7d08b85 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..ba6aa5e7492
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+makedepends="freetype-devel imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+export LDFLAGS="-L${XBPS_CROSS_BASE} -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}	

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (38 preceding siblings ...)
  2021-01-08 22:03 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 22:05 ` kawaiiamber
  2021-01-08 22:06 ` ndowens
                   ` (39 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 22:05 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1180 bytes --]

From 8f40ef50e2c8456ca39855df524f9b7c5fdb8a8d Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..627a27deffd
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+makedepends="freetype-devel imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}	

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (39 preceding siblings ...)
  2021-01-08 22:05 ` kawaiiamber
@ 2021-01-08 22:06 ` ndowens
  2021-01-08 22:07 ` kawaiiamber
                   ` (38 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 22:06 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 246 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757022422

Comment:
Of course you will need to fix lint, but lets make sure build is fine, locally it cross-compiled this way

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (40 preceding siblings ...)
  2021-01-08 22:06 ` ndowens
@ 2021-01-08 22:07 ` kawaiiamber
  2021-01-08 22:08 ` ndowens
                   ` (37 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 22:07 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 537 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757022971

Comment:
Done, thanks for the help @ndowens !! So far I've just been reading the Manual for void-packages. However, I wonder if there are other good sources to learn about VOID packaging. e.g. the only difference between `hostmakedeps` and `makedeps` is one is for "target" and one is for "host". However, I'm not too sure what this itself means. Also, the Lint check is detecting trailing whitespace?

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (41 preceding siblings ...)
  2021-01-08 22:07 ` kawaiiamber
@ 2021-01-08 22:08 ` ndowens
  2021-01-08 22:08 ` ndowens
                   ` (36 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 22:08 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 654 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757023306

Comment:
> Done, thanks for the help @ndowens !! So far I've just been reading the Manual for void-packages. However, I wonder if there are other good sources to learn about VOID packaging. e.g. the only difference between `hostmakedeps` and `makedeps` is one is for "target" and one is for "host". However, I'm not too sure what this itself means. Also, the Lint check is detecting trailing whitespace?

Most -devel goes in makedepends, though sometimes gettext-devel will need to go in hostmakedepends for cross-builds

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (42 preceding siblings ...)
  2021-01-08 22:08 ` ndowens
@ 2021-01-08 22:08 ` ndowens
  2021-01-08 22:10 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (35 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ndowens @ 2021-01-08 22:08 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 692 bytes --]

New comment by ndowens on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757023306

Comment:
> Done, thanks for the help @ndowens !! So far I've just been reading the Manual for void-packages. However, I wonder if there are other good sources to learn about VOID packaging. e.g. the only difference between `hostmakedeps` and `makedeps` is one is for "target" and one is for "host". However, I'm not too sure what this itself means. Also, the Lint check is detecting trailing whitespace?

Most -devel goes in makedepends, though sometimes gettext-devel will need to go in hostmakedepends for cross-builds

Now all you need to do is fix lint

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (43 preceding siblings ...)
  2021-01-08 22:08 ` ndowens
@ 2021-01-08 22:10 ` kawaiiamber
  2021-01-08 22:10 ` kawaiiamber
                   ` (34 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 22:10 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1179 bytes --]

From 080ad9556c31ce68868fce74235bce062da5c2ce Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..f793e8fc63c
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+makedepends="freetype-devel imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (44 preceding siblings ...)
  2021-01-08 22:10 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 22:10 ` kawaiiamber
  2021-01-08 22:17 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (33 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 22:10 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 215 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757023949

Comment:
Think I got it, there was a tab character after the last curley brace.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (45 preceding siblings ...)
  2021-01-08 22:10 ` kawaiiamber
@ 2021-01-08 22:17 ` kawaiiamber
  2021-01-08 22:19 ` kawaiiamber
                   ` (32 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 22:17 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.4.1
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1164 bytes --]

From cd92bd2613d1233d30ab72b2af04b01aee6661da Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.4.1

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..e16aa6b011c
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.4.1
+revision=1
+build_style=gnu-makefile
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=34afe623055b12aa64c0d870c4d55f5594ae45041b571ded660cea84ba3cc40e
+
+export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.4.1
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (46 preceding siblings ...)
  2021-01-08 22:17 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-08 22:19 ` kawaiiamber
  2021-01-09 13:03 ` [PR PATCH] [Updated] New package: xmenu-4.5.0 kawaiiamber
                   ` (31 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-08 22:19 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 185 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757027285

Comment:
Removed `freetype-devel` from `makedeps`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (47 preceding siblings ...)
  2021-01-08 22:19 ` kawaiiamber
@ 2021-01-09 13:03 ` kawaiiamber
  2021-01-09 13:04 ` kawaiiamber
                   ` (30 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-09 13:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.0
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1164 bytes --]

From b06dccafc165b78411830833bef992148d68ddcc Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.0

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..0b8c72a9467
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.0
+revision=1
+build_style=gnu-makefile
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="https://github.com/phillbush/xmenu/archive/v${version}.tar.gz"
+checksum=50a3968101f9cb5fd55cbdc872475225e98d4412cd96bfea1449185a72cb49c5
+
+export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (48 preceding siblings ...)
  2021-01-09 13:03 ` [PR PATCH] [Updated] New package: xmenu-4.5.0 kawaiiamber
@ 2021-01-09 13:04 ` kawaiiamber
  2021-01-09 13:06 ` kawaiiamber
                   ` (29 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-09 13:04 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 257 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757166848

Comment:
Updated to version 4.5.0, which has been release [here](https://github.com/phillbush/xmenu/releases/tag/v4.5.0).

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (49 preceding siblings ...)
  2021-01-09 13:04 ` kawaiiamber
@ 2021-01-09 13:06 ` kawaiiamber
  2021-01-09 18:56 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (28 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-09 13:06 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 258 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757166848

Comment:
Updated to version 4.5.0, which has been released [here](https://github.com/phillbush/xmenu/releases/tag/v4.5.0).

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (50 preceding siblings ...)
  2021-01-09 13:06 ` kawaiiamber
@ 2021-01-09 18:56 ` kawaiiamber
  2021-01-09 22:21 ` phillbush
                   ` (27 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-09 18:56 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.0
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1141 bytes --]

From 3feaed615bd22aff663dc0e1ca7d55a590feb98f Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.0

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..e1848f1dedc
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.0
+revision=1
+build_style=gnu-makefile
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=50a3968101f9cb5fd55cbdc872475225e98d4412cd96bfea1449185a72cb49c5
+
+export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (51 preceding siblings ...)
  2021-01-09 18:56 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-09 22:21 ` phillbush
  2021-01-09 22:22 ` phillbush
                   ` (26 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: phillbush @ 2021-01-09 22:21 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 292 bytes --]

New comment by phillbush on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757376163

Comment:
Just to notify that I released a new patched [version](https://github.com/phillbush/xmenu/releases/tag/v4.5.1) with a small bugfix for the Shift key.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (52 preceding siblings ...)
  2021-01-09 22:21 ` phillbush
@ 2021-01-09 22:22 ` phillbush
  2021-01-09 22:26 ` phillbush
                   ` (25 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: phillbush @ 2021-01-09 22:22 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 353 bytes --]

New comment by phillbush on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757376163

Comment:
Just to notify that I released a new patched [version](https://github.com/phillbush/xmenu/releases/tag/v4.5.1) with a small bugfix for a bug introduced in the type-to-select feature added in the latest version.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (53 preceding siblings ...)
  2021-01-09 22:22 ` phillbush
@ 2021-01-09 22:26 ` phillbush
  2021-01-09 22:29 ` phillbush
                   ` (24 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: phillbush @ 2021-01-09 22:26 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 353 bytes --]

New comment by phillbush on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757376163

Comment:
Just to notify that I released a new patched [version](https://github.com/phillbush/xmenu/releases/tag/v4.5.2) with a small bugfix for a bug introduced in the type-to-select feature added in the latest version.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (54 preceding siblings ...)
  2021-01-09 22:26 ` phillbush
@ 2021-01-09 22:29 ` phillbush
  2021-01-09 22:41 ` phillbush
                   ` (23 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: phillbush @ 2021-01-09 22:29 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 353 bytes --]

New comment by phillbush on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757376163

Comment:
Just to notify that I released a new patched [version](https://github.com/phillbush/xmenu/releases/tag/v4.5.3) with a small bugfix for a bug introduced in the type-to-select feature added in the latest version.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (55 preceding siblings ...)
  2021-01-09 22:29 ` phillbush
@ 2021-01-09 22:41 ` phillbush
  2021-01-09 22:55 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (22 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: phillbush @ 2021-01-09 22:41 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 353 bytes --]

New comment by phillbush on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757376163

Comment:
Just to notify that I released a new patched [version](https://github.com/phillbush/xmenu/releases/tag/v4.5.4) with a small bugfix for a bug introduced in the type-to-select feature added in the latest version.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (56 preceding siblings ...)
  2021-01-09 22:41 ` phillbush
@ 2021-01-09 22:55 ` kawaiiamber
  2021-01-09 22:55 ` kawaiiamber
                   ` (21 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-09 22:55 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.0
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1141 bytes --]

From d334893cf59cd4a38fe85422c482c9bb3cfe7a57 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.4

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..94e8f2e9998
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.4
+revision=1
+build_style=gnu-makefile
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=5c11dde7e5b07d1abd8fe338a264eec1a121a633f6b27e2bcef8baa4cebb82c9
+
+export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.0
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (57 preceding siblings ...)
  2021-01-09 22:55 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-01-09 22:55 ` kawaiiamber
  2021-01-18  2:55 ` [PR PATCH] [Updated] New package: xmenu-4.5.4 kawaiiamber
                   ` (20 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-09 22:55 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 173 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-757379924

Comment:
Updated to use version 4.5.4

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (58 preceding siblings ...)
  2021-01-09 22:55 ` kawaiiamber
@ 2021-01-18  2:55 ` kawaiiamber
  2021-01-18  2:56 ` kawaiiamber
                   ` (19 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-18  2:55 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.4
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1141 bytes --]

From 9bde227283b8a8310928bf9e13e5af3d78ba466c Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.4

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..94e8f2e9998
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.4
+revision=1
+build_style=gnu-makefile
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=5c11dde7e5b07d1abd8fe338a264eec1a121a633f6b27e2bcef8baa4cebb82c9
+
+export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (59 preceding siblings ...)
  2021-01-18  2:55 ` [PR PATCH] [Updated] New package: xmenu-4.5.4 kawaiiamber
@ 2021-01-18  2:56 ` kawaiiamber
  2021-01-18  2:59 ` kawaiiamber
                   ` (18 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-18  2:56 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.4
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1178 bytes --]

From e29344e5c3359cce1f4be661b4cac9b77bff169b Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.4

---
 srcpkgs/xmenu/template | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..9dd0f690fcc
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,21 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.4
+revision=1
+build_style=gnu-makefile
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=5c11dde7e5b07d1abd8fe338a264eec1a121a633f6b27e2bcef8baa4cebb82c9
+
+if [ ${CROSS_BUILD} ]; then
+	export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+	CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+fi
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (60 preceding siblings ...)
  2021-01-18  2:56 ` kawaiiamber
@ 2021-01-18  2:59 ` kawaiiamber
  2021-04-18  4:58 ` [PR REVIEW] " ericonr
                   ` (17 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-01-18  2:59 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.4
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1141 bytes --]

From 2f19d11fee9aeed7a230fd6342254015e5c818b0 Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.4

---
 srcpkgs/xmenu/template | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 00000000000..94e8f2e9998
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,19 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.4
+revision=1
+build_style=gnu-makefile
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=5c11dde7e5b07d1abd8fe338a264eec1a121a633f6b27e2bcef8baa4cebb82c9
+
+export LDFLAGS="-L${XBPS_CROSS_BASE}/usr/lib -lfontconfig -lXft -lX11 -lXinerama -lImlib2"
+CFLAGS="-I${XBPS_CROSS_BASE}/usr/include/freetype2"
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (61 preceding siblings ...)
  2021-01-18  2:59 ` kawaiiamber
@ 2021-04-18  4:58 ` ericonr
  2021-04-18  4:58 ` ericonr
                   ` (16 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ericonr @ 2021-04-18  4:58 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 159 bytes --]

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r615341438

Comment:
4.5.5 is out

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (62 preceding siblings ...)
  2021-04-18  4:58 ` [PR REVIEW] " ericonr
@ 2021-04-18  4:58 ` ericonr
  2021-04-20  0:37 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (15 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ericonr @ 2021-04-18  4:58 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 422 bytes --]

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r615342068

Comment:
LDFLAGS doesn't need to be exported. But there's a possibly simpler way of fixing this without those variables.

`make_build_args="LOCALINC=$XBPS_CROSS_BASE/usr/include LOCALLIB=$XBPS_CROSS_BASE/usr/lib FREETYPEINC=$XBPS_CROSS_BASE/usr/include/freetype2 LDFLAGS+=\$(LIBS)"`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (63 preceding siblings ...)
  2021-04-18  4:58 ` ericonr
@ 2021-04-20  0:37 ` kawaiiamber
  2021-04-20  0:37 ` [PR REVIEW] " kawaiiamber
                   ` (14 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-04-20  0:37 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.4
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1154 bytes --]

From cbcbe9e8d0bbeef3c1ce968f1c657cb7d3973f6b Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.5

---
 srcpkgs/xmenu/template | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 000000000000..2de62752947f
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,17 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.5
+revision=1
+build_style=gnu-makefile
+make_build_args="LOCALINC=$XBPS_CROSS_BASE/usr/include LOCALLIB=$XBPS_CROSS_BASE/usr/lib FREETYPEINC=$XBPS_CROSS_BASE/usr/include/freetype2 LDFLAGS+=\$(LIBS)"
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=13fda8068d3886a590365d19d6ff84f04c0a4c9a516d79569000514067b4e77b
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (64 preceding siblings ...)
  2021-04-20  0:37 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-04-20  0:37 ` kawaiiamber
  2021-04-20  0:38 ` kawaiiamber
                   ` (13 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-04-20  0:37 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 188 bytes --]

New review comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r616264742

Comment:
Updated version and checksum to 4.5.5

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (65 preceding siblings ...)
  2021-04-20  0:37 ` [PR REVIEW] " kawaiiamber
@ 2021-04-20  0:38 ` kawaiiamber
  2021-04-20  3:30 ` ericonr
                   ` (12 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-04-20  0:38 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 180 bytes --]

New review comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r616264910

Comment:
Did vars in `make_build_args`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (66 preceding siblings ...)
  2021-04-20  0:38 ` kawaiiamber
@ 2021-04-20  3:30 ` ericonr
  2021-04-21 23:42 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (11 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ericonr @ 2021-04-20  3:30 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 247 bytes --]

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r616317281

Comment:
I knew I was forgetting something `CFLAGS+=\$(INCS)`

Also, please break that line at ~80 columns.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (67 preceding siblings ...)
  2021-04-20  3:30 ` ericonr
@ 2021-04-21 23:42 ` kawaiiamber
  2021-04-21 23:42 ` [PR REVIEW] " kawaiiamber
                   ` (10 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-04-21 23:42 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.4
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1186 bytes --]

From 22401cb2820ba2154550a6ccdddcdc8b85b2c0bd Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.5

---
 srcpkgs/xmenu/template | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 000000000000..71f72cbd4aa7
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,20 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.5
+revision=1
+build_style=gnu-makefile
+make_build_args="LOCALINC=$XBPS_CROSS_BASE/usr/include \
+	LOCALLIB=$XBPS_CROSS_BASE/usr/lib \
+	FREETYPEINC=$XBPS_CROSS_BASE/usr/include/freetype2 \
+	LDFLAGS+=\$(LIBS) CFLAGS+=\$(INCS)"
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=13fda8068d3886a590365d19d6ff84f04c0a4c9a516d79569000514067b4e77b
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (68 preceding siblings ...)
  2021-04-21 23:42 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-04-21 23:42 ` kawaiiamber
  2021-04-22  0:21 ` ericonr
                   ` (9 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-04-21 23:42 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 201 bytes --]

New review comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r617965386

Comment:
Added `CFLAGS+=\$(INCS)` and broke at ~80 columns.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (69 preceding siblings ...)
  2021-04-21 23:42 ` [PR REVIEW] " kawaiiamber
@ 2021-04-22  0:21 ` ericonr
  2021-04-22  1:11 ` [PR PATCH] [Updated] " kawaiiamber
                   ` (8 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: ericonr @ 2021-04-22  0:21 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 413 bytes --]

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r617978449

Comment:
```suggestion
make_build_args="LOCALINC=$XBPS_CROSS_BASE/usr/include
 LOCALLIB=$XBPS_CROSS_BASE/usr/lib
 FREETYPEINC=$XBPS_CROSS_BASE/usr/include/freetype2
 LDFLAGS+=\$(LIBS) CFLAGS+=\$(INCS)"
```

No need to use backslashes. And indentation should be spaces.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Updated] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (70 preceding siblings ...)
  2021-04-22  0:21 ` ericonr
@ 2021-04-22  1:11 ` kawaiiamber
  2021-04-22  1:12 ` [PR REVIEW] " kawaiiamber
                   ` (7 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-04-22  1:11 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

There is an updated pull request by kawaiiamber against master on the void-packages repository

https://github.com/kawaiiamber/void-packages xmenu
https://github.com/void-linux/void-packages/pull/27763

New package: xmenu-4.5.4
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

A patch file from https://github.com/void-linux/void-packages/pull/27763.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xmenu-27763.patch --]
[-- Type: text/x-diff, Size: 1180 bytes --]

From 81f88f21e1383e44cae405befeaffc4a08cd1cfc Mon Sep 17 00:00:00 2001
From: KawaiiAmber <japaneselearning101@gmail.com>
Date: Fri, 8 Jan 2021 08:41:49 -0700
Subject: [PATCH] New package: xmenu-4.5.5

---
 srcpkgs/xmenu/template | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100644 srcpkgs/xmenu/template

diff --git a/srcpkgs/xmenu/template b/srcpkgs/xmenu/template
new file mode 100644
index 000000000000..3b825805af0b
--- /dev/null
+++ b/srcpkgs/xmenu/template
@@ -0,0 +1,20 @@
+# Template file for 'xmenu'
+pkgname=xmenu
+version=4.5.5
+revision=1
+build_style=gnu-makefile
+make_build_args="LOCALINC=$XBPS_CROSS_BASE/usr/include
+ LOCALLIB=$XBPS_CROSS_BASE/usr/lib
+ FREETYPEINC=$XBPS_CROSS_BASE/usr/include/freetype2
+ LDFLAGS+=\$(LIBS) CFLAGS+=\$(INCS)"
+makedepends="imlib2-devel libX11-devel libXft-devel libXinerama-devel"
+short_desc="X11 menu utility"
+maintainer="KawaiiAmber <japaneselearning101@gmail.com>"
+license="MIT"
+homepage="https://github.com/phillbush/xmenu"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=13fda8068d3886a590365d19d6ff84f04c0a4c9a516d79569000514067b4e77b
+
+post_install() {
+	vlicense LICENSE
+}

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR REVIEW] New package: xmenu-4.5.4
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (71 preceding siblings ...)
  2021-04-22  1:11 ` [PR PATCH] [Updated] " kawaiiamber
@ 2021-04-22  1:12 ` kawaiiamber
  2021-06-20 15:33 ` New package: xmenu-4.5.5 kawaiiamber
                   ` (6 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-04-22  1:12 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 208 bytes --]

New review comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#discussion_r617994119

Comment:
Got it ~ Removed the `\`'s and replaced tabs with spaces.

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.5
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (72 preceding siblings ...)
  2021-04-22  1:12 ` [PR REVIEW] " kawaiiamber
@ 2021-06-20 15:33 ` kawaiiamber
  2021-06-20 23:41 ` kawaiiamber
                   ` (5 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-06-20 15:33 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 665 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-864572039

Comment:
> Is there a reason for this not to get merged? I am thinking about switching to void, but I really need xmenu...

A member must verify that it meets the requirements for inclusion into the package repository. VOID generally only accepts stable / released packages. In the mean time, you can use `xbps-src` from the template to package it locally yourself - no need to wait if you want to use VOID. Just clone my branch (or copy template) and `xbps-src pkg xmenu` then `xbps-install --repository=/path/to/package.xbps`

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.5
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (73 preceding siblings ...)
  2021-06-20 15:33 ` New package: xmenu-4.5.5 kawaiiamber
@ 2021-06-20 23:41 ` kawaiiamber
  2021-06-21 15:20 ` jzbor
                   ` (4 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: kawaiiamber @ 2021-06-20 23:41 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 757 bytes --]

New comment by kawaiiamber on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-864572039

Comment:
> Is there a reason for this not to get merged? I am thinking about switching to void, but I really need xmenu...

A member must verify that it meets the requirements for inclusion into the package repository. VOID generally only accepts stable / released packages. In the mean time, you can use `xbps-src` from the template to package it locally yourself - no need to wait if you want to use VOID. Just clone my branch (or copy template) and `xbps-src pkg xmenu` then `xbps-install --repository=/path/to/package.xbps`. You'd help by testing it out. I've tested it out myself but the more people the better ^-^

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.5
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (74 preceding siblings ...)
  2021-06-20 23:41 ` kawaiiamber
@ 2021-06-21 15:20 ` jzbor
  2021-06-21 15:21 ` jzbor
                   ` (3 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: jzbor @ 2021-06-21 15:20 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 748 bytes --]

New comment by jzbor on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-865120953

Comment:
Ok so I tried it and get the following error:

```
=> ca-certificates-20200601_2: installing host dependencies: libressl-3.1.5_2 ...
=> ca-certificates-20200601_2: running do-fetch hook: 00-distfiles ...
=> ca-certificates-20200601_2: fetching distfile 'ca-certificates_20200601.tar.xz'...
http://ftp.debian.org/debian/pool/main/c/ca-certificates/ca-certificates_20200601.tar.xz: Not Found
=> ERROR: ca-certificates-20200601_2: failed to fetch ca-certificates_20200601.tar.xz.
```
I attached the whole log as well.
[output.txt](https://github.com/void-linux/void-packages/files/6687731/output.txt)


^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.5
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (75 preceding siblings ...)
  2021-06-21 15:20 ` jzbor
@ 2021-06-21 15:21 ` jzbor
  2021-06-21 15:33 ` paper42
                   ` (2 subsequent siblings)
  79 siblings, 0 replies; 81+ messages in thread
From: jzbor @ 2021-06-21 15:21 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 785 bytes --]

New comment by jzbor on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-865120953

Comment:
Ok so I tried it and get the following error, when running `./xbps-src pkg xmenu`:

```
=> ca-certificates-20200601_2: installing host dependencies: libressl-3.1.5_2 ...
=> ca-certificates-20200601_2: running do-fetch hook: 00-distfiles ...
=> ca-certificates-20200601_2: fetching distfile 'ca-certificates_20200601.tar.xz'...
http://ftp.debian.org/debian/pool/main/c/ca-certificates/ca-certificates_20200601.tar.xz: Not Found
=> ERROR: ca-certificates-20200601_2: failed to fetch ca-certificates_20200601.tar.xz.
```
I attached the whole log as well.
[output.txt](https://github.com/void-linux/void-packages/files/6687731/output.txt)


^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.5
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (76 preceding siblings ...)
  2021-06-21 15:21 ` jzbor
@ 2021-06-21 15:33 ` paper42
  2021-06-21 16:01 ` jzbor
  2021-06-23 13:28 ` [PR PATCH] [Closed]: " ericonr
  79 siblings, 0 replies; 81+ messages in thread
From: paper42 @ 2021-06-21 15:33 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 188 bytes --]

New comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-865130156

Comment:
@jzbor you will have to rebase on top of master

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: New package: xmenu-4.5.5
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (77 preceding siblings ...)
  2021-06-21 15:33 ` paper42
@ 2021-06-21 16:01 ` jzbor
  2021-06-23 13:28 ` [PR PATCH] [Closed]: " ericonr
  79 siblings, 0 replies; 81+ messages in thread
From: jzbor @ 2021-06-21 16:01 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 236 bytes --]

New comment by jzbor on void-packages repository

https://github.com/void-linux/void-packages/pull/27763#issuecomment-865155687

Comment:
> @jzbor you will have to rebase on top of master

Ah thanks that helped. It installs fine now!

^ permalink raw reply	[flat|nested] 81+ messages in thread

* Re: [PR PATCH] [Closed]: New package: xmenu-4.5.5
  2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
                   ` (78 preceding siblings ...)
  2021-06-21 16:01 ` jzbor
@ 2021-06-23 13:28 ` ericonr
  79 siblings, 0 replies; 81+ messages in thread
From: ericonr @ 2021-06-23 13:28 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 819 bytes --]

There's a closed pull request on the void-packages repository

New package: xmenu-4.5.5
https://github.com/void-linux/void-packages/pull/27763

Description:
# Current issues
When running `./xbps-src pkg xmenu`, it seems that it's looking for `ft2build.h`, even though `freetype-devel` isn't actually a dependency to build. Even when including `freetype-devel`, behaviour doesn't seem to change. It is specifically complaining about line 15: `#include <X11/Xft/Xft.h>`. I'm not too sure why it's complaining about not being able to find `ft2build.h`. The only dependency I included that the author seems to forget to mention is `libXinerama-devel` as on line 16 of `xmenu.c` it seems to call for it: `#include <X11/extensions/Xinerama.h>`. I am able to build and install the app from source successfully on my machine.

^ permalink raw reply	[flat|nested] 81+ messages in thread

end of thread, other threads:[~2021-06-23 13:28 UTC | newest]

Thread overview: 81+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-08 16:00 [PR PATCH] New package: xmenu-4.4.1 kawaiiamber
2021-01-08 16:02 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 17:06 ` ericonr
2021-01-08 17:06 ` ericonr
2021-01-08 17:07 ` ericonr
2021-01-08 17:16 ` kawaiiamber
2021-01-08 17:19 ` kawaiiamber
2021-01-08 17:20 ` kawaiiamber
2021-01-08 17:36 ` kawaiiamber
2021-01-08 17:41 ` kawaiiamber
2021-01-08 17:47 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 17:55 ` kawaiiamber
2021-01-08 17:59 ` kawaiiamber
2021-01-08 18:11 ` kawaiiamber
2021-01-08 18:14 ` kawaiiamber
2021-01-08 18:22 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 18:26 ` kawaiiamber
2021-01-08 18:31 ` kawaiiamber
2021-01-08 18:34 ` kawaiiamber
2021-01-08 18:36 ` kawaiiamber
2021-01-08 20:32 ` [PR REVIEW] " ndowens
2021-01-08 20:38 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 20:38 ` [PR REVIEW] " kawaiiamber
2021-01-08 20:43 ` ndowens
2021-01-08 20:44 ` ndowens
2021-01-08 20:50 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 20:50 ` kawaiiamber
2021-01-08 20:51 ` ndowens
2021-01-08 20:52 ` kawaiiamber
2021-01-08 20:53 ` ndowens
2021-01-08 20:53 ` kawaiiamber
2021-01-08 20:54 ` kawaiiamber
2021-01-08 20:54 ` kawaiiamber
2021-01-08 20:56 ` kawaiiamber
2021-01-08 21:51 ` kawaiiamber
2021-01-08 22:00 ` ndowens
2021-01-08 22:00 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 22:03 ` ndowens
2021-01-08 22:03 ` ndowens
2021-01-08 22:03 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 22:05 ` kawaiiamber
2021-01-08 22:06 ` ndowens
2021-01-08 22:07 ` kawaiiamber
2021-01-08 22:08 ` ndowens
2021-01-08 22:08 ` ndowens
2021-01-08 22:10 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 22:10 ` kawaiiamber
2021-01-08 22:17 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-08 22:19 ` kawaiiamber
2021-01-09 13:03 ` [PR PATCH] [Updated] New package: xmenu-4.5.0 kawaiiamber
2021-01-09 13:04 ` kawaiiamber
2021-01-09 13:06 ` kawaiiamber
2021-01-09 18:56 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-09 22:21 ` phillbush
2021-01-09 22:22 ` phillbush
2021-01-09 22:26 ` phillbush
2021-01-09 22:29 ` phillbush
2021-01-09 22:41 ` phillbush
2021-01-09 22:55 ` [PR PATCH] [Updated] " kawaiiamber
2021-01-09 22:55 ` kawaiiamber
2021-01-18  2:55 ` [PR PATCH] [Updated] New package: xmenu-4.5.4 kawaiiamber
2021-01-18  2:56 ` kawaiiamber
2021-01-18  2:59 ` kawaiiamber
2021-04-18  4:58 ` [PR REVIEW] " ericonr
2021-04-18  4:58 ` ericonr
2021-04-20  0:37 ` [PR PATCH] [Updated] " kawaiiamber
2021-04-20  0:37 ` [PR REVIEW] " kawaiiamber
2021-04-20  0:38 ` kawaiiamber
2021-04-20  3:30 ` ericonr
2021-04-21 23:42 ` [PR PATCH] [Updated] " kawaiiamber
2021-04-21 23:42 ` [PR REVIEW] " kawaiiamber
2021-04-22  0:21 ` ericonr
2021-04-22  1:11 ` [PR PATCH] [Updated] " kawaiiamber
2021-04-22  1:12 ` [PR REVIEW] " kawaiiamber
2021-06-20 15:33 ` New package: xmenu-4.5.5 kawaiiamber
2021-06-20 23:41 ` kawaiiamber
2021-06-21 15:20 ` jzbor
2021-06-21 15:21 ` jzbor
2021-06-21 15:33 ` paper42
2021-06-21 16:01 ` jzbor
2021-06-23 13:28 ` [PR PATCH] [Closed]: " ericonr

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).