Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] growlight: update to 1.2.32.
@ 2021-04-08 19:41 mobinmob
  2021-04-09  2:32 ` ericonr
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: mobinmob @ 2021-04-08 19:41 UTC (permalink / raw)
  To: ml

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

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

https://github.com/mobinmob/void-packages growlight
https://github.com/void-linux/void-packages/pull/30095

growlight: update to 1.2.32.
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From 45a2bc56e2c714a13568495aaf09107b899fd12a Mon Sep 17 00:00:00 2001
From: mobinmob <mobinmob@disroot.org>
Date: Thu, 8 Apr 2021 22:40:01 +0300
Subject: [PATCH] growlight: update to 1.2.32.

---
 srcpkgs/growlight/patches/doctest-2.4.6.patch | 74 -------------------
 srcpkgs/growlight/template                    |  4 +-
 2 files changed, 2 insertions(+), 76 deletions(-)
 delete mode 100644 srcpkgs/growlight/patches/doctest-2.4.6.patch

diff --git a/srcpkgs/growlight/patches/doctest-2.4.6.patch b/srcpkgs/growlight/patches/doctest-2.4.6.patch
deleted file mode 100644
index c307ea9d5ef4..000000000000
--- a/srcpkgs/growlight/patches/doctest-2.4.6.patch
+++ /dev/null
@@ -1,74 +0,0 @@
-Index: growlight-1.2.31/tests/gpt.cpp
-===================================================================
---- growlight-1.2.31.orig/tests/gpt.cpp
-+++ growlight-1.2.31/tests/gpt.cpp
-@@ -6,6 +6,9 @@
- 
- #define UUID "\x5E\x86\x90\xEF\xD0\x30\x03\x46\x99\x3D\x54\x6E\xB0\xE7\x1B\x0D"
- 
-+template <typename T>
-+T xnormalise(T value) { return value; }
-+
- TEST_CASE("GPT") {
- 
-   // First eight bytes must be "EFI PART"
-@@ -19,14 +22,14 @@ TEST_CASE("GPT") {
-   SUBCASE("Revision") {
-     gpt_header head;
-     CHECK(0 == initialize_gpt(&head, 512, 4194287, 34, nullptr));
--    CHECK(0x00010000 == head.revision);
-+    CHECK(0x00010000 == xnormalise(head.revision));
-   }
- 
-   // Bytes 0xc--0xf must be >= 92, should be the logical block size
-   SUBCASE("HeaderSize") {
-     gpt_header head;
-     CHECK(0 == initialize_gpt(&head, 512, 4194287, 34, nullptr));
--    CHECK(92 == head.headsize);
-+    CHECK(92 == xnormalise(head.headsize));
-   }
- 
-   // Bytes 0x18--0x1f are the sector of the GPT primary, usually 1
-@@ -34,8 +37,8 @@ TEST_CASE("GPT") {
-   SUBCASE("GPTLBAs") {
-     gpt_header head;
-     CHECK(0 == initialize_gpt(&head, 512, 100000, 34, nullptr));
--    CHECK(1 == head.lba);
--    CHECK(100000 == head.backuplba);
-+    CHECK(1 == xnormalise(head.lba));
-+    CHECK(100000 == xnormalise(head.backuplba));
-   }
- 
-   // Verify the 16-byte UUID is as specified
-@@ -61,17 +64,17 @@ TEST_CASE("GPT") {
-     gpt_header head;
-     CHECK(0 == initialize_gpt(&head, 512, 4194287, 34, UUID));
-     // partition entry size must be a positive multiple of 128 (usually 128)
--    CHECK(0 < head.partsize);
-+    CHECK(0 < xnormalise(head.partsize));
-     CHECK(0 == (head.partsize % 128));
-     // number of partition entries, usually 128 (MINIMUM_GPT_ENTRIES)
--    CHECK(128 <= head.partcount);
-+    CHECK(128 <= xnormalise(head.partcount));
-     auto entries = new gpt_entry[head.partcount];
-     memset(entries, 0, sizeof(*entries) * head.partcount);
-     CHECK(0 == update_crc(&head, entries));
-     // FIXME fix on big-endian!
--    WARN(2006165414 == head.crc);
--    CHECK(0 == head.reserved);
--    CHECK(2874462854 == head.partcrc);
-+    WARN(2006165414 == xnormalise(head.crc));
-+    CHECK(0 == xnormalise(head.reserved));
-+    CHECK(2874462854 == xnormalise(head.partcrc));
-     delete[] entries;
-   }
- 
-@@ -85,7 +88,7 @@ TEST_CASE("GPT") {
-     memset(sector, 0xff, sizeof(sector));
-     gpt_header* head = reinterpret_cast<gpt_header*>(sector);
-     CHECK(0 == initialize_gpt(head, sizeof(sector), 4194287, 34, nullptr));
--    CHECK(92 == head->headsize);
-+    CHECK(92 == xnormalise(head->headsize));
-     for(size_t idx = 92 ; idx < sizeof(sector) ; ++idx){
-       CHECK(0 == sector[idx]);
-     }
diff --git a/srcpkgs/growlight/template b/srcpkgs/growlight/template
index f47402c51221..4456db1f8185 100644
--- a/srcpkgs/growlight/template
+++ b/srcpkgs/growlight/template
@@ -1,6 +1,6 @@
 # Template file for 'growlight'
 pkgname=growlight
-version=1.2.31
+version=1.2.32
 revision=1
 build_style=cmake
 configure_args="$(vopt_bool zfs USE_LIBZFS) $(vopt_bool man USE_PANDOC)"
@@ -14,7 +14,7 @@ maintainer="mobinmob <mobinmob@disroot.org>"
 license="GPL-3.0-or-later"
 homepage="https://nick-black.com/dankwiki/index.php/Growlight"
 distfiles="https://github.com/dankamongmen/growlight/archive/v${version}.tar.gz"
-checksum=25cf643d99be88d299756ccb2933868641abecbc26793f5d87cfae93a461e2d6
+checksum=f15357602d04e10a34d5d3f0d83a66a328638ce40c7b20ffaeac68459318edf2
 patch_args=-Np1
 
 build_options="man zfs"

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

* Re: growlight: update to 1.2.32.
  2021-04-08 19:41 [PR PATCH] growlight: update to 1.2.32 mobinmob
@ 2021-04-09  2:32 ` ericonr
  2021-04-09  8:57 ` mobinmob
  2021-04-09 13:01 ` [PR PATCH] [Merged]: " ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: ericonr @ 2021-04-09  2:32 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/30095#issuecomment-816357770

Comment:
How does this match up with #30016 ?

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

* Re: growlight: update to 1.2.32.
  2021-04-08 19:41 [PR PATCH] growlight: update to 1.2.32 mobinmob
  2021-04-09  2:32 ` ericonr
@ 2021-04-09  8:57 ` mobinmob
  2021-04-09 13:01 ` [PR PATCH] [Merged]: " ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: mobinmob @ 2021-04-09  8:57 UTC (permalink / raw)
  To: ml

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

New comment by mobinmob on void-packages repository

https://github.com/void-linux/void-packages/pull/30095#issuecomment-816533637

Comment:
`growlight` works with both notcurses v. 2.2.4 and 2.2.5 without any problems. I have opened an issue (https://github.com/dankamongmen/notcurses/issues/1506) on the upstream project for https://github.com/void-linux/void-packages/pull/30016 and a fix is already commited to master. `growlight` never had issues - it does not use bitmaps afaict.

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

* Re: [PR PATCH] [Merged]: growlight: update to 1.2.32.
  2021-04-08 19:41 [PR PATCH] growlight: update to 1.2.32 mobinmob
  2021-04-09  2:32 ` ericonr
  2021-04-09  8:57 ` mobinmob
@ 2021-04-09 13:01 ` ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: ericonr @ 2021-04-09 13:01 UTC (permalink / raw)
  To: ml

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

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

growlight: update to 1.2.32.
https://github.com/void-linux/void-packages/pull/30095

Description:
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [X] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

end of thread, other threads:[~2021-04-09 13:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-08 19:41 [PR PATCH] growlight: update to 1.2.32 mobinmob
2021-04-09  2:32 ` ericonr
2021-04-09  8:57 ` mobinmob
2021-04-09 13:01 ` [PR PATCH] [Merged]: " 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).