Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] New package: macchina-0.9.2
@ 2021-06-08 12:24 machadofguilherme
  2021-06-08 18:08 ` [PR PATCH] [Closed]: " Chocimier
                   ` (13 more replies)
  0 siblings, 14 replies; 15+ messages in thread
From: machadofguilherme @ 2021-06-08 12:24 UTC (permalink / raw)
  To: ml

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

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

https://github.com/machadofguilherme/void-packages macchina
https://github.com/void-linux/void-packages/pull/31361

New package: macchina-0.9.2
<!-- 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?
- [ ] 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/31361.patch is attached

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

From 67c62ebc047665c0d063737c169171916f27c218 Mon Sep 17 00:00:00 2001
From: Guilherme Machado <machadofguilherme@gmail.com>
Date: Tue, 8 Jun 2021 09:23:35 -0300
Subject: [PATCH] New package: macchina-0.9.2

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

diff --git a/srcpkgs/macchina/template b/srcpkgs/macchina/template
new file mode 100644
index 000000000000..02fcab9bd9a9
--- /dev/null
+++ b/srcpkgs/macchina/template
@@ -0,0 +1,23 @@
+# Template file for 'macchina'
+pkgname=macchina
+version=0.9.2
+revision=1
+archs="x86_64"
+build_style=cargo
+hostmakedepends="cargo rust"
+makedepends="wmctrl"
+depends="libgcc glibc"
+short_desc="System information with emphasis on minimalism"
+maintainer="Guilherme Machado <machadofguilherme@gmail.com>"
+license="MIT"
+homepage="crates.io/crates/macchina"
+distfiles="https://github.com/Macchina-CLI/${pkgname}/archive/v${version}.tar.gz"
+checksum=f5ac11a49470639d81424b208fc5d95b111a77b97138af7fce347ad3daae8005
+
+post_install() {
+	vmkdir usr/share/doc/${pkgname}
+	vdoc README.md
+	vdoc CHANGELOG.txt
+	vlicense LICENSE
+	rm -rf ${DESTDIR}/usr/lib*
+}

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

* Re: [PR PATCH] [Closed]: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
@ 2021-06-08 18:08 ` Chocimier
  2021-06-08 18:08 ` Chocimier
                   ` (12 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: Chocimier @ 2021-06-08 18:08 UTC (permalink / raw)
  To: ml

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

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

New package: macchina-0.9.2
https://github.com/void-linux/void-packages/pull/31361

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

#### General
- [x] 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] 15+ messages in thread

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
  2021-06-08 18:08 ` [PR PATCH] [Closed]: " Chocimier
@ 2021-06-08 18:08 ` Chocimier
  2021-06-08 20:36 ` machadofguilherme
                   ` (11 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: Chocimier @ 2021-06-08 18:08 UTC (permalink / raw)
  To: ml

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

New comment by Chocimier on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-856983431

Comment:
> In particular, new themes are highly unlikely to be accepted

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
  2021-06-08 18:08 ` [PR PATCH] [Closed]: " Chocimier
  2021-06-08 18:08 ` Chocimier
@ 2021-06-08 20:36 ` machadofguilherme
  2021-06-08 21:23 ` q66
                   ` (10 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: machadofguilherme @ 2021-06-08 20:36 UTC (permalink / raw)
  To: ml

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

New comment by machadofguilherme on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-857114380

Comment:
> > In particular, new themes are highly unlikely to be accepted

That's not a theme. You closed this PR without even checking the content.

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (2 preceding siblings ...)
  2021-06-08 20:36 ` machadofguilherme
@ 2021-06-08 21:23 ` q66
  2021-06-08 21:24 ` q66
                   ` (9 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: q66 @ 2021-06-08 21:23 UTC (permalink / raw)
  To: ml

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

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-857165575

Comment:
it's <yet another of the many tools to help make fancy screencaps> so it might as well be the same thing 

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (3 preceding siblings ...)
  2021-06-08 21:23 ` q66
@ 2021-06-08 21:24 ` q66
  2021-06-17 14:22 ` eoli3n
                   ` (8 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: q66 @ 2021-06-08 21:24 UTC (permalink / raw)
  To: ml

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

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-857165575

Comment:
it's yet another of the many tools to help make fancy screencaps so it might as well be the same thing 

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (4 preceding siblings ...)
  2021-06-08 21:24 ` q66
@ 2021-06-17 14:22 ` eoli3n
  2021-06-17 14:41 ` q66
                   ` (7 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: eoli3n @ 2021-06-17 14:22 UTC (permalink / raw)
  To: ml

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

New comment by eoli3n on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863283514

Comment:
That's not just to make "fancy screenshots". It could be a kind of mix between conky and neofetch to run it at each login on servers for exemple. To get useful informations, like available upgrades, zfs pool storage percent, etc...
I asked the dev, he liked the idea : https://github.com/Macchina-CLI/macchina/issues/105

So, please reconsider this, I would co-maintain it.
ping @q66 @Chocimier 

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (5 preceding siblings ...)
  2021-06-17 14:22 ` eoli3n
@ 2021-06-17 14:41 ` q66
  2021-06-17 14:56 ` eoli3n
                   ` (6 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: q66 @ 2021-06-17 14:41 UTC (permalink / raw)
  To: ml

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

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863299571

Comment:
just install it with cargo if you want it, that's what rust people want you to do anyway

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (6 preceding siblings ...)
  2021-06-17 14:41 ` q66
@ 2021-06-17 14:56 ` eoli3n
  2021-06-17 15:11 ` grtcdr
                   ` (5 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: eoli3n @ 2021-06-17 14:56 UTC (permalink / raw)
  To: ml

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

New comment by eoli3n on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863311302

Comment:
python packages needs pip, ruby needs gems, but we wrap those.
If I would compile the package myself, I would use Gentoo.

```bash
void-packages/srcpkgs on master  
➜ grep -ri 'build_style=cargo' | wc -l
134
```

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (7 preceding siblings ...)
  2021-06-17 14:56 ` eoli3n
@ 2021-06-17 15:11 ` grtcdr
  2021-06-17 15:49 ` q66
                   ` (4 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: grtcdr @ 2021-06-17 15:11 UTC (permalink / raw)
  To: ml

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

New comment by grtcdr on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863324200

Comment:
@q66 So the package was rejected because it mimics other similar programs?

The open source community has been making clones of everything since its inception, but it would be nice if you considered what macchina does differently from its alternatives before quickly dismissing it.

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (8 preceding siblings ...)
  2021-06-17 15:11 ` grtcdr
@ 2021-06-17 15:49 ` q66
  2021-06-17 16:15 ` grtcdr
                   ` (3 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: q66 @ 2021-06-17 15:49 UTC (permalink / raw)
  To: ml

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

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863355995

Comment:
FWIW, considering rust's overall hostility against distribution packaging (no way to manage dependencies outside of cargo, no dynamic linking of rust libraries, no reasonable way to control fetching of distfiles and so on) i believe the majority of these other rust programs in the repository shouldn't have been packaged either (other distributions in general do not package them)

if there's something we definitely do not need, it's a multitude of *fetch programs that all serve the purpose of making fancy screenshots for /r/unixporn

also fwiw, a program with 158 dependencies and 7.5MB binary size can hardly be considered minimalistic (and regardless of the functionality it implements, it's ridiculous for a *fetch program) and literally noone cares about whether a fetch takes 10ms or 150ms (not to mention that benchmarking against what is literally a shell script - neofetch - is pretty amusing)

so yeah, i still don't see a reason to package every single rust program under the sun especially considering the rust project seemingly does not care for providing reasonable means of packaging them

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (9 preceding siblings ...)
  2021-06-17 15:49 ` q66
@ 2021-06-17 16:15 ` grtcdr
  2021-06-17 16:20 ` grtcdr
                   ` (2 subsequent siblings)
  13 siblings, 0 replies; 15+ messages in thread
From: grtcdr @ 2021-06-17 16:15 UTC (permalink / raw)
  To: ml

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

New comment by grtcdr on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863376096

Comment:
I just read the quality requirements for a package to be accepted, and with all due respect, I don't see how your subjective opinion on the matter is a criteria that must be met.

You seem to have a problem with Rust in general, and I really can't help you with that.

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (10 preceding siblings ...)
  2021-06-17 16:15 ` grtcdr
@ 2021-06-17 16:20 ` grtcdr
  2021-06-17 16:23 ` ahesford
  2021-06-17 16:24 ` Chocimier
  13 siblings, 0 replies; 15+ messages in thread
From: grtcdr @ 2021-06-17 16:20 UTC (permalink / raw)
  To: ml

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

New comment by grtcdr on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863376096

Comment:
I just read the quality requirements for a package to be accepted, and with all due respect, I don't see how your subjective opinion on the matter is a criterion that must be met.

You seem to have a problem with Rust in general, and I really can't help you with that.

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (11 preceding siblings ...)
  2021-06-17 16:20 ` grtcdr
@ 2021-06-17 16:23 ` ahesford
  2021-06-17 16:24 ` Chocimier
  13 siblings, 0 replies; 15+ messages in thread
From: ahesford @ 2021-06-17 16:23 UTC (permalink / raw)
  To: ml

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

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863381995

Comment:
It should be exceedingly obvious that getting subjective approval for a new package from at least one member with the authority to approve pull requests is a criterion that must be met. This package failed to gain that approval. At least three project members agree that the value of this package is offset by the maintenance burden. This matter is closed.

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

* Re: New package: macchina-0.9.2
  2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
                   ` (12 preceding siblings ...)
  2021-06-17 16:23 ` ahesford
@ 2021-06-17 16:24 ` Chocimier
  13 siblings, 0 replies; 15+ messages in thread
From: Chocimier @ 2021-06-17 16:24 UTC (permalink / raw)
  To: ml

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

New comment by Chocimier on void-packages repository

https://github.com/void-linux/void-packages/pull/31361#issuecomment-863382751

Comment:
macchina really advertises itself like nothing more than uname's theme (and new themes are not accepted). libmacchina may be nice but is not build by that template, and has no known consumers. Running arbitrary command is not yet there. Add some killer feature to it and come back.

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

end of thread, other threads:[~2021-06-17 16:24 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-08 12:24 [PR PATCH] New package: macchina-0.9.2 machadofguilherme
2021-06-08 18:08 ` [PR PATCH] [Closed]: " Chocimier
2021-06-08 18:08 ` Chocimier
2021-06-08 20:36 ` machadofguilherme
2021-06-08 21:23 ` q66
2021-06-08 21:24 ` q66
2021-06-17 14:22 ` eoli3n
2021-06-17 14:41 ` q66
2021-06-17 14:56 ` eoli3n
2021-06-17 15:11 ` grtcdr
2021-06-17 15:49 ` q66
2021-06-17 16:15 ` grtcdr
2021-06-17 16:20 ` grtcdr
2021-06-17 16:23 ` ahesford
2021-06-17 16:24 ` Chocimier

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).