Github messages for voidlinux
 help / color / mirror / Atom feed
From: classabbyamp <classabbyamp@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: New package: logisim-evolution-3.8.0
Date: Tue, 17 Jan 2023 19:16:12 +0100	[thread overview]
Message-ID: <20230117181612.5fCQlPph8hOixjw1JWMNQrRveQ80uCOfg6xXarPNxXk@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-41603@inbox.vuxu.org>

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/41603#issuecomment-1385836929

Comment:
to summarise:
```patch
From 528ba136392986688f1bfb7c4327fcf9125e1035 Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Tue, 17 Jan 2023 13:15:15 -0500
Subject: [PATCH] pr review


diff --git a/srcpkgs/logisim-evolution/files/logisim-evolution b/srcpkgs/logisim-evolution/files/logisim-evolution
index 1039e777fb..7fdaedabdd 100644
--- a/srcpkgs/logisim-evolution/files/logisim-evolution
+++ b/srcpkgs/logisim-evolution/files/logisim-evolution
@@ -1,6 +1,4 @@
 #!/bin/sh
 
 export JAVA_HOME=/usr/lib/jvm/openjdk17
-$JAVA_HOME/bin/java -jar \
-       /usr/lib/@PKGNAME@/logisim-evolution-@VERSION@-all.jar $@
-
+$JAVA_HOME/bin/java -jar /usr/lib/logisim-evolution/logisim-evolution.jar $@
diff --git a/srcpkgs/logisim-evolution/template b/srcpkgs/logisim-evolution/template
index b59cd3c716..b89cacbb82 100644
--- a/srcpkgs/logisim-evolution/template
+++ b/srcpkgs/logisim-evolution/template
@@ -18,14 +18,11 @@ do_build() {
 }
 
 do_install() {
-       vinstall build/libs/logisim-evolution-${version}-all.jar 0644 usr/lib/${pkgname}
+       vinstall build/libs/logisim-evolution-${version}-all.jar 0644 usr/lib/logisim-evolution logisim-evolution.jar
 
        vbin ${FILESDIR}/logisim-evolution
-       sed -i -e "s/@VERSION@/${version}/g" -e "s/@PKGNAME@/${pkgname}/g" \
-               ${DESTDIR}/usr/bin/logisim-evolution
 
-       vinstall artwork/logisim-evolution-icon.svg 0644 \
-               usr/share/icons/hicolor/apps com.github.reds.LogisimEvolution.svg
-       vinstall support/Flatpak/com.github.reds.LogisimEvolution.desktop 0644 \
-               usr/share/applications
+       vinstall artwork/logisim-evolution-icon.svg 0644 usr/share/icons/hicolor/scalable/apps com.github.reds.LogisimEvolution.svg
+       vinstall support/Flatpak/com.github.reds.LogisimEvolution.desktop 0644 usr/share/applications
+       vinstall support/Flatpak/com.github.reds.LogisimEvolution.desktop 0644 usr/share/mime/packages
 }
-- 
2.39.0
```

  parent reply	other threads:[~2023-01-17 18:16 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-12 17:03 [PR PATCH] " voidbert
2023-01-12 17:16 ` [PR REVIEW] " classabbyamp
2023-01-12 17:16 ` classabbyamp
2023-01-12 17:16 ` classabbyamp
2023-01-12 17:22 ` voidbert
2023-01-12 21:24 ` [PR PATCH] [Updated] " voidbert
2023-01-12 22:24 ` voidbert
2023-01-12 22:34 ` voidbert
2023-01-13  0:49 ` classabbyamp
2023-01-13  1:10 ` [PR REVIEW] " classabbyamp
2023-01-13  1:10 ` classabbyamp
2023-01-13  1:10 ` classabbyamp
2023-01-13  1:10 ` classabbyamp
2023-01-13  1:10 ` classabbyamp
2023-01-13  1:10 ` classabbyamp
2023-01-13  1:10 ` classabbyamp
2023-01-13 15:27 ` [PR PATCH] [Updated] " voidbert
2023-01-13 15:38 ` voidbert
2023-01-17 18:10 ` [PR REVIEW] " classabbyamp
2023-01-17 18:10 ` classabbyamp
2023-01-17 18:10 ` classabbyamp
2023-01-17 18:10 ` classabbyamp
2023-01-17 18:10 ` classabbyamp
2023-01-17 18:16 ` classabbyamp [this message]
2023-01-18 12:46 ` [PR PATCH] [Updated] " voidbert
2023-01-18 20:12 ` [PR REVIEW] " classabbyamp
2023-01-18 22:35 ` [PR PATCH] [Updated] " voidbert
2023-01-18 22:36 ` [PR REVIEW] " voidbert
2023-01-18 23:33 ` [PR PATCH] [Merged]: " classabbyamp

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=20230117181612.5fCQlPph8hOixjw1JWMNQrRveQ80uCOfg6xXarPNxXk@z \
    --to=classabbyamp@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).