unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#70407] [PATCH] gnu: Add fnc
@ 2024-04-15 20:25 ashish.is--- via Guix-patches via
  2024-04-15 22:11 ` [bug#70407] QA review for 70407 Daniel Ziltener via Guix-patches via
  2024-04-16 22:21 ` bug#70407: [PATCH] gnu: Add fnc Christopher Baines
  0 siblings, 2 replies; 5+ messages in thread
From: ashish.is--- via Guix-patches via @ 2024-04-15 20:25 UTC (permalink / raw)
  To: 70407; +Cc: Ashish SHUKLA

From: Ashish SHUKLA <ashish.is@lostca.se>

* gnu/packages/version-control.scm (fnc): New variable.

Change-Id: Idf1cb254c3ae18bacb86bf020511a5d34e2d3684
Signed-off-by: Ashish SHUKLA <ashish.is@lostca.se>
---
 gnu/packages/version-control.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index 96ca1df552..33607940ec 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -4145,3 +4145,28 @@ (define-public git-sizer
     (description "Compute various size metrics for a Git repository, flagging
 those that might cause problems or inconvenience.")
     (license license:expat)))
+
+(define-public fnc
+  (package
+    (name "fnc")
+    (version "0.16")
+    (source (origin
+              (method url-fetch)
+              (uri
+               (string-append "https://fnc.bsdbox.org/uv/dl/fnc-" version ".tar.gz"))
+              (sha256
+               (base32
+                "1npnbdz5i4p61ri76vx6awggbc0q19y8b26l3sy4wxmaxkly7gwy"))))
+    (build-system gnu-build-system)
+    (arguments
+     '(#:phases (modify-phases %standard-phases
+                  (delete 'configure))
+       #:tests? #f
+       #:make-flags (list "CC=gcc"
+			  (string-append "PREFIX=" (assoc-ref %outputs "out")))))
+    (inputs (list ncurses zlib sqlite-next))
+    (home-page "https://fnc.bsdbox.org")
+    (synopsis "Interactive text-based user interface for Fossil")
+    (description "fnc uses ncurses and libfossil to create a fossil ui experience in
+the terminal, and parse local changes at the hunk level to prepare atomic commits.")
+    (license license:isc)))

base-commit: a8353e9d6b34fd8d42d2e8f14ce844849fe9c293
-- 
2.41.0





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

* [bug#70407] QA review for 70407
  2024-04-15 20:25 [bug#70407] [PATCH] gnu: Add fnc ashish.is--- via Guix-patches via
@ 2024-04-15 22:11 ` Daniel Ziltener via Guix-patches via
  2024-04-16 22:21 ` bug#70407: [PATCH] gnu: Add fnc Christopher Baines
  1 sibling, 0 replies; 5+ messages in thread
From: Daniel Ziltener via Guix-patches via @ 2024-04-15 22:11 UTC (permalink / raw)
  To: control, 70407

user guix
usertag 70407 + reviewed-looks-good
thanks

Guix QA review form submission:


Items marked as checked: New package licenses, New package synopsis and 
descriptions




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

* bug#70407: [PATCH] gnu: Add fnc
  2024-04-15 20:25 [bug#70407] [PATCH] gnu: Add fnc ashish.is--- via Guix-patches via
  2024-04-15 22:11 ` [bug#70407] QA review for 70407 Daniel Ziltener via Guix-patches via
@ 2024-04-16 22:21 ` Christopher Baines
  2024-04-17  9:25   ` [bug#70407] Re: [bug#70407] " Ashish via Guix-patches via
  1 sibling, 1 reply; 5+ messages in thread
From: Christopher Baines @ 2024-04-16 22:21 UTC (permalink / raw)
  To: ashish.is--- via Guix-patches via, Daniel Ziltener; +Cc: 70407-done, ashish.is

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

"ashish.is--- via Guix-patches" via <guix-patches@gnu.org> writes:

> From: Ashish SHUKLA <ashish.is@lostca.se>
>
> * gnu/packages/version-control.scm (fnc): New variable.
>
> Change-Id: Idf1cb254c3ae18bacb86bf020511a5d34e2d3684
> Signed-off-by: Ashish SHUKLA <ashish.is@lostca.se>
> ---
>  gnu/packages/version-control.scm | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>
> diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
> index 96ca1df552..33607940ec 100644
> --- a/gnu/packages/version-control.scm
> +++ b/gnu/packages/version-control.scm
> @@ -4145,3 +4145,28 @@ (define-public git-sizer
>      (description "Compute various size metrics for a Git repository, flagging
>  those that might cause problems or inconvenience.")
>      (license license:expat)))
> +
> +(define-public fnc
> +  (package
> +    (name "fnc")
> +    (version "0.16")
> +    (source (origin
> +              (method url-fetch)
> +              (uri
> +               (string-append "https://fnc.bsdbox.org/uv/dl/fnc-" version ".tar.gz"))
> +              (sha256
> +               (base32
> +                "1npnbdz5i4p61ri76vx6awggbc0q19y8b26l3sy4wxmaxkly7gwy"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     '(#:phases (modify-phases %standard-phases
> +                  (delete 'configure))
> +       #:tests? #f
> +       #:make-flags (list "CC=gcc"
> +			  (string-append "PREFIX=" (assoc-ref %outputs "out")))))

The arguments needed a bit of touching up. Using gexp's is the new
style. It's unclear why the tests are disabled, I guess it's because
there are no tests so I've added a comment to this effect. Hardcoding
CC=gcc also causes problems for cross building, so use (cc-for-target)
instead.

> +    (inputs (list ncurses zlib sqlite-next))
> +    (home-page "https://fnc.bsdbox.org")
> +    (synopsis "Interactive text-based user interface for Fossil")
> +    (description "fnc uses ncurses and libfossil to create a fossil ui experience in
> +the terminal, and parse local changes at the hunk level to prepare atomic commits.")
> +    (license license:isc)))
>
> base-commit: a8353e9d6b34fd8d42d2e8f14ce844849fe9c293

I did have a quick check over the source, including a concatenated
sqlite seems unusual and not ideal, but I'm not quite sure how it's
used.

One final change I made was moving the package definition up off the
bottom of the file (as if everyone adds new packages there, it causes
merge conflicts).

I've pushed this to master with the changes I describe as
a45b2c7c63c6af9c42698e12b6809c41c0926de3 so please take a look.

Thanks,

Chris

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

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

* [bug#70407] Re: [bug#70407] [PATCH] gnu: Add fnc
  2024-04-16 22:21 ` bug#70407: [PATCH] gnu: Add fnc Christopher Baines
@ 2024-04-17  9:25   ` Ashish via Guix-patches via
  2024-04-17 10:51     ` Christopher Baines
  0 siblings, 1 reply; 5+ messages in thread
From: Ashish via Guix-patches via @ 2024-04-17  9:25 UTC (permalink / raw)
  To: Christopher Baines; +Cc: dziltener, 70407-done, 70407

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

On Tue, Apr 16, 2024 at 11:21:26PM +0100, Christopher Baines wrote:
> "ashish.is--- via Guix-patches" via <guix-patches@gnu.org> writes:
> 
> > From: Ashish SHUKLA <ashish.is@lostca.se>
> >
> > * gnu/packages/version-control.scm (fnc): New variable.
> >
> > Change-Id: Idf1cb254c3ae18bacb86bf020511a5d34e2d3684
> > Signed-off-by: Ashish SHUKLA <ashish.is@lostca.se>
> > ---
> >  gnu/packages/version-control.scm | 25 +++++++++++++++++++++++++
> >  1 file changed, 25 insertions(+)
> >
> > diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
> > index 96ca1df552..33607940ec 100644
> > --- a/gnu/packages/version-control.scm
> > +++ b/gnu/packages/version-control.scm
> > @@ -4145,3 +4145,28 @@ (define-public git-sizer
> >      (description "Compute various size metrics for a Git repository, flagging
> >  those that might cause problems or inconvenience.")
> >      (license license:expat)))
> > +
> > +(define-public fnc
> > +  (package
> > +    (name "fnc")
> > +    (version "0.16")
> > +    (source (origin
> > +              (method url-fetch)
> > +              (uri
> > +               (string-append "https://fnc.bsdbox.org/uv/dl/fnc-" version ".tar.gz"))
> > +              (sha256
> > +               (base32
> > +                "1npnbdz5i4p61ri76vx6awggbc0q19y8b26l3sy4wxmaxkly7gwy"))))
> > +    (build-system gnu-build-system)
> > +    (arguments
> > +     '(#:phases (modify-phases %standard-phases
> > +                  (delete 'configure))
> > +       #:tests? #f
> > +       #:make-flags (list "CC=gcc"
> > +			  (string-append "PREFIX=" (assoc-ref %outputs "out")))))
> 
> The arguments needed a bit of touching up. Using gexp's is the new
> style. It's unclear why the tests are disabled, I guess it's because
> there are no tests so I've added a comment to this effect. Hardcoding
> CC=gcc also causes problems for cross building, so use (cc-for-target)
> instead.

Thanks for catching this, it indeed crossed my mind, but I was not sure what to replace it with should someone chose to build with `clang' (or other C language compiler), and `cc' is not present.

> > +    (inputs (list ncurses zlib sqlite-next))
> > +    (home-page "https://fnc.bsdbox.org")
> > +    (synopsis "Interactive text-based user interface for Fossil")
> > +    (description "fnc uses ncurses and libfossil to create a fossil ui experience in
> > +the terminal, and parse local changes at the hunk level to prepare atomic commits.")
> > +    (license license:isc)))
> >
> > base-commit: a8353e9d6b34fd8d42d2e8f14ce844849fe9c293
> 
> I did have a quick check over the source, including a concatenated
> sqlite seems unusual and not ideal, but I'm not quite sure how it's
> used.

libfossil is a library implementation of fossil ships with bundled sqlite3, and the author decide to bundle libfossil, and so sqlite3 comes with it.

> 
> One final change I made was moving the package definition up off the
> bottom of the file (as if everyone adds new packages there, it causes
> merge conflicts).
> 
> I've pushed this to master with the changes I describe as
> a45b2c7c63c6af9c42698e12b6809c41c0926de3 so please take a look.
> 
> Thanks,
> 
> Chris

Thanks!
-- 
Ashish | GPG: F682 CDCC 39DC 0FEA E116  20B6 C746 CFA9 E74F A4B0

"If I destroy you, what business is it of yours ?" (Dark Forest, Liu Cixin)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 963 bytes --]

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

* [bug#70407] [PATCH] gnu: Add fnc
  2024-04-17  9:25   ` [bug#70407] Re: [bug#70407] " Ashish via Guix-patches via
@ 2024-04-17 10:51     ` Christopher Baines
  0 siblings, 0 replies; 5+ messages in thread
From: Christopher Baines @ 2024-04-17 10:51 UTC (permalink / raw)
  To: Ashish; +Cc: dziltener, 70407-done, 70407

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

Ashish <ashish.is@lostca.se> writes:

> On Tue, Apr 16, 2024 at 11:21:26PM +0100, Christopher Baines wrote:
>> "ashish.is--- via Guix-patches" via <guix-patches@gnu.org> writes:
>>
>> > +    (inputs (list ncurses zlib sqlite-next))
>> > +    (home-page "https://fnc.bsdbox.org")
>> > +    (synopsis "Interactive text-based user interface for Fossil")
>> > +    (description "fnc uses ncurses and libfossil to create a fossil ui experience in
>> > +the terminal, and parse local changes at the hunk level to prepare atomic commits.")
>> > +    (license license:isc)))
>> >
>> > base-commit: a8353e9d6b34fd8d42d2e8f14ce844849fe9c293
>>
>> I did have a quick check over the source, including a concatenated
>> sqlite seems unusual and not ideal, but I'm not quite sure how it's
>> used.
>
> libfossil is a library implementation of fossil ships with bundled
> sqlite3, and the author decide to bundle libfossil, and so sqlite3
> comes with it.

Right, it would be good to package libfossil and unbundle sqlite3 there,
then use that libfossil package here. If libfossil can't use the built
sqlite3 from Guix, then maybe the Guix package source can be used.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

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

end of thread, other threads:[~2024-04-17 10:52 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-15 20:25 [bug#70407] [PATCH] gnu: Add fnc ashish.is--- via Guix-patches via
2024-04-15 22:11 ` [bug#70407] QA review for 70407 Daniel Ziltener via Guix-patches via
2024-04-16 22:21 ` bug#70407: [PATCH] gnu: Add fnc Christopher Baines
2024-04-17  9:25   ` [bug#70407] Re: [bug#70407] " Ashish via Guix-patches via
2024-04-17 10:51     ` Christopher Baines

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).