Github messages for voidlinux
 help / color / mirror / Atom feed
From: ajtcode71 <ajtcode71@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] New package: yabridgectl-4.0.2
Date: Mon, 26 Sep 2022 12:22:52 +0200	[thread overview]
Message-ID: <20220926102252.-I4e6gLTCP4W6-se_h3brOOhQ3gTXF_iftEuVIJaYfk@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-39476@inbox.vuxu.org>

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

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

https://github.com/ajtcode71/void-packages yabridgectl
https://github.com/void-linux/void-packages/pull/39476

New package: yabridgectl-4.0.2
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**


#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements):  
- 
xlint check complains about the wrksrc not being top level.  The source of yabridgectl is distributed within the yabridge source and is several levels down. Compiles and works well by keeping the origin source in the correct locations and having the multilevel wrksrc set.


#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)


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

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

From 1477f818885ebed251ff813ce001b0e26fa1b625 Mon Sep 17 00:00:00 2001
From: Anthony Thompson <athompson@posteo.net>
Date: Mon, 26 Sep 2022 18:07:23 +1000
Subject: [PATCH 1/2] yabridgectl templated added

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

diff --git a/srcpkgs/yabridgectl/template b/srcpkgs/yabridgectl/template
new file mode 100644
index 000000000000..c3e3126e66b9
--- /dev/null
+++ b/srcpkgs/yabridgectl/template
@@ -0,0 +1,16 @@
+# Template file for 'yabridgectl'
+pkgname=yabridgectl
+version=4.0.2
+revision=1
+archs="x86_64"
+## software developer has this package source as part of the yabridge source tree.
+## Yabridgectl is several levels down. unable to comply with xlint for top level directory
+wrksrc="yabridge-${version}/tools/yabridgectl"
+build_style=cargo
+hostmakedepends="grep"
+short_desc="Yabridgectl oontrols yabridge configurations"
+maintainer="Anthony Thompson <athompson@posteo.net>"
+license="GPL-3.0-or-later"
+homepage="https://github.com/robbert-vdh/yabridge"
+distfiles="https://github.com/robbert-vdh/yabridge/archive/refs/tags/${version}.tar.gz"
+checksum="86fcd65b6f3dd5cf60c2afa476f63959a2cc4ef4d4629810916739d7af645f01"

From 5bfff755a7dbf9ad11237d2bafe1b77fdce29c28 Mon Sep 17 00:00:00 2001
From: Anthony Thompson <athompson@posteo.net>
Date: Mon, 26 Sep 2022 20:22:12 +1000
Subject: [PATCH 2/2] fixed xlint error. added buildwrksrc

---
 srcpkgs/yabridgectl/template | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/yabridgectl/template b/srcpkgs/yabridgectl/template
index c3e3126e66b9..219c921edd42 100644
--- a/srcpkgs/yabridgectl/template
+++ b/srcpkgs/yabridgectl/template
@@ -3,9 +3,8 @@ pkgname=yabridgectl
 version=4.0.2
 revision=1
 archs="x86_64"
-## software developer has this package source as part of the yabridge source tree.
-## Yabridgectl is several levels down. unable to comply with xlint for top level directory
-wrksrc="yabridge-${version}/tools/yabridgectl"
+wrksrc="yabridge-${version}"
+build_wrksrc="tools/yabridgectl"
 build_style=cargo
 hostmakedepends="grep"
 short_desc="Yabridgectl oontrols yabridge configurations"

  parent reply	other threads:[~2022-09-26 10:22 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-26  8:17 [PR PATCH] " ajtcode71
2022-09-26  8:39 ` paper42
2022-09-26  8:40 ` [PR REVIEW] " paper42
2022-09-26 10:22 ` ajtcode71 [this message]
2022-09-26 11:08 ` paper42
2022-09-26 11:10 ` paper42
2022-09-26 23:38 ` [PR PATCH] [Updated] " ajtcode71
2022-09-26 23:45 ` ajtcode71
2022-09-27  0:35 ` [PR REVIEW] " paper42
2022-10-01  8:10 ` [PR PATCH] [Updated] " ajtcode71
2022-10-01  8:16 ` ajtcode71
2022-10-01  8:19 ` ajtcode71
2022-10-01  8:25 ` ajtcode71
2022-10-01 10:57 ` [PR REVIEW] " paper42
2022-12-31  1:55 ` github-actions
2023-01-15  2:00 ` [PR PATCH] [Closed]: " github-actions

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220926102252.-I4e6gLTCP4W6-se_h3brOOhQ3gTXF_iftEuVIJaYfk@z \
    --to=ajtcode71@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).