* [bug#63900] [PATCH] gnu: Add emacs-fb2-reader.
@ 2023-06-04 16:30 Adam Kandur
2023-06-05 16:15 ` Nicolas Goaziou
2023-06-10 10:38 ` Adam Kandur
0 siblings, 2 replies; 5+ messages in thread
From: Adam Kandur @ 2023-06-04 16:30 UTC (permalink / raw)
To: 63900; +Cc: Adam Kandur
* gnu/packages/emacs-xyz.scm (emacs-fb2-reader): New variable.
---
gnu/packages/emacs-xyz.scm | 32 ++++++++++++++++++++++++++++++++
1 file changed, 32 insertions(+)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index b77bd0aa83..f563063d7f 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -1632,6 +1632,38 @@ (define-public emacs-fzf
purpose finder.")
(license license:gpl3+))))
+(define-public emacs-fb2-reader
+ (let ((revision "0")
+ ;; named branch is outdated
+ (commit "9836db284749e0cef4c43c2cb5358c82ae9b8589"))
+ (package
+ (name "emacs-fb2-reader")
+ (version (git-version "0.1" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/jumper047/fb2-reader")
+ (commit commit)))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32 "0vx4b9wnmx1bng8wak5r7yryyvgib4m46l6b877xzkdhsjr3rbsi"))))
+ (build-system emacs-build-system)
+ (native-inputs
+ (list emacs-buttercup
+ emacs-dash
+ emacs-s
+ emacs-async))
+ (propagated-inputs
+ (list emacs-visual-fill-column
+ emacs-f
+ unzip))
+ (home-page "https://github.com/jumper047/fb2-reader")
+ (synopsis "Emacs plugin to read .fb2 and .fb2.zip ebooks")
+ (description "This package allows users to read fb2 books
+whithout quiting Emacs.")
+ (license license:gpl3))))
+
(define-public emacs-minions
(package
(name "emacs-minions")
--
2.40.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [bug#63900] [PATCH] gnu: Add emacs-fb2-reader.
2023-06-04 16:30 [bug#63900] [PATCH] gnu: Add emacs-fb2-reader Adam Kandur
@ 2023-06-05 16:15 ` Nicolas Goaziou
2023-06-06 20:35 ` Adam
2023-06-10 10:38 ` Adam Kandur
1 sibling, 1 reply; 5+ messages in thread
From: Nicolas Goaziou @ 2023-06-05 16:15 UTC (permalink / raw)
To: Adam Kandur; +Cc: 63900
Hello,
Adam Kandur <4164616d.4b616e647572@gmail.com> writes:
> * gnu/packages/emacs-xyz.scm (emacs-fb2-reader): New variable.
Thank you. Some comments follow.
> +(define-public emacs-fb2-reader
> + (let ((revision "0")
> + ;; named branch is outdated
> + (commit "9836db284749e0cef4c43c2cb5358c82ae9b8589"))
This exactly matches version 0.1.1 so you should add "; version bump"
comment at the end of the line, and use 0.1.1 as the version.
> + (native-inputs
> + (list emacs-buttercup
> + emacs-dash
> + emacs-s
> + emacs-async))
There are tests, could you try running them?
Note that inputs should be ordered alphabetically.
> + (propagated-inputs
> + (list emacs-visual-fill-column
> + emacs-f
> + unzip))
Likewise.
Also, unzip should be an input rather than a propagated input, and
"fb2-reader.el" should be patched accordingly (i.e., replacing
"unzip"
with
(search-input-files inputs "/bin/unzip"))
> + (home-page "https://github.com/jumper047/fb2-reader")
> + (synopsis "Emacs plugin to read .fb2 and .fb2.zip ebooks")
I suggest:
Emacs plugin to read FictionBook2 ebooks
> + (description "This package allows users to read fb2 books
> +whithout quiting Emacs.")
What about:
FB2 Reader provides a major mode for reading FictionBook2 (@file{.fb2}
and @file{.fb2.zip} files) ebooks.
> + (license license:gpl3))))
License is actually gpl3+, as mentioned in the main file.
Could you send an updated patch?
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 5+ messages in thread
* [bug#63900] [PATCH] gnu: Add emacs-fb2-reader.
2023-06-05 16:15 ` Nicolas Goaziou
@ 2023-06-06 20:35 ` Adam
0 siblings, 0 replies; 5+ messages in thread
From: Adam @ 2023-06-06 20:35 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: 63900
Sure, I will try make a new patch this week
On Mon, Jun 5, 2023 at 7:15 PM Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
>
> Hello,
>
> Adam Kandur <4164616d.4b616e647572@gmail.com> writes:
>
> > * gnu/packages/emacs-xyz.scm (emacs-fb2-reader): New variable.
>
> Thank you. Some comments follow.
>
> > +(define-public emacs-fb2-reader
> > + (let ((revision "0")
> > + ;; named branch is outdated
> > + (commit "9836db284749e0cef4c43c2cb5358c82ae9b8589"))
>
> This exactly matches version 0.1.1 so you should add "; version bump"
> comment at the end of the line, and use 0.1.1 as the version.
>
> > + (native-inputs
> > + (list emacs-buttercup
> > + emacs-dash
> > + emacs-s
> > + emacs-async))
>
> There are tests, could you try running them?
>
> Note that inputs should be ordered alphabetically.
>
> > + (propagated-inputs
> > + (list emacs-visual-fill-column
> > + emacs-f
> > + unzip))
>
> Likewise.
>
> Also, unzip should be an input rather than a propagated input, and
> "fb2-reader.el" should be patched accordingly (i.e., replacing
>
> "unzip"
>
> with
>
> (search-input-files inputs "/bin/unzip"))
>
> > + (home-page "https://github.com/jumper047/fb2-reader")
> > + (synopsis "Emacs plugin to read .fb2 and .fb2.zip ebooks")
>
> I suggest:
>
> Emacs plugin to read FictionBook2 ebooks
>
> > + (description "This package allows users to read fb2 books
> > +whithout quiting Emacs.")
>
> What about:
>
> FB2 Reader provides a major mode for reading FictionBook2 (@file{.fb2}
> and @file{.fb2.zip} files) ebooks.
>
> > + (license license:gpl3))))
>
> License is actually gpl3+, as mentioned in the main file.
>
> Could you send an updated patch?
>
> Regards,
> --
> Nicolas Goaziou
^ permalink raw reply [flat|nested] 5+ messages in thread
* [bug#63900] [PATCH] gnu: Add emacs-fb2-reader.
2023-06-04 16:30 [bug#63900] [PATCH] gnu: Add emacs-fb2-reader Adam Kandur
2023-06-05 16:15 ` Nicolas Goaziou
@ 2023-06-10 10:38 ` Adam Kandur
2023-06-10 18:46 ` bug#63900: " Nicolas Goaziou
1 sibling, 1 reply; 5+ messages in thread
From: Adam Kandur @ 2023-06-10 10:38 UTC (permalink / raw)
To: mail; +Cc: Adam Kandur, 63900
* gnu/packages/emacs-xyz.scm (emacs-fb2-reader): New variable.
---
gnu/packages/emacs-xyz.scm | 44 ++++++++++++++++++++++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index b77bd0a..5335b3f 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -2365,6 +2365,50 @@ (define-public emacs-flycheck
provides an optional IDE-like error list.")
(license license:gpl3+))) ;+GFDLv1.3+ for the manual
+(define-public emacs-fb2-reader
+ (let ((revision "0")
+ (commit "9836db284749e0cef4c43c2cb5358c82ae9b8589")) ; version bump
+ (package
+ (name "emacs-fb2-reader")
+ (version (git-version "0.1.1" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/jumper047/fb2-reader")
+ (commit commit)))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32 "0vx4b9wnmx1bng8wak5r7yryyvgib4m46l6b877xzkdhsjr3rbsi"))))
+ (build-system emacs-build-system)
+ (arguments
+ (list
+ #:tests? #t
+ #:test-command
+ #~(list "emacs" "-Q" "--batch" "-L" "."
+ "--eval" "(load-file \"tests/test-fb2-reader.el\")")
+ #:phases
+ #~(modify-phases %standard-phases
+ (add-after 'unpack 'qualify-paths
+ (lambda* (#:key inputs #:allow-other-keys)
+ (let ((unzip (search-input-file inputs "/bin/unzip")))
+ (substitute* "fb2-reader.el"
+ (("unzip") unzip))))))))
+ (inputs (list unzip))
+ (native-inputs
+ (list emacs-buttercup
+ emacs-dash
+ emacs-s
+ emacs-async))
+ (propagated-inputs
+ (list emacs-visual-fill-column
+ emacs-f))
+ (home-page "https://github.com/jumper047/fb2-reader")
+ (synopsis "Emacs plugin to read FictionBook2 ebooks")
+ (description "FB2 Reader provides a major mode for reading FictionBook2 (@file{.fb2}
+and @file{.fb2.zip} files) ebooks.")
+ (license license:gpl3+))))
+
(define-public emacs-flymake-collection
(package
(name "emacs-flymake-collection")
--
2.40.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* bug#63900: [PATCH] gnu: Add emacs-fb2-reader.
2023-06-10 10:38 ` Adam Kandur
@ 2023-06-10 18:46 ` Nicolas Goaziou
0 siblings, 0 replies; 5+ messages in thread
From: Nicolas Goaziou @ 2023-06-10 18:46 UTC (permalink / raw)
To: Adam Kandur; +Cc: 63900-done
Hello,
Adam Kandur <4164616d.4b616e647572@gmail.com> writes:
> * gnu/packages/emacs-xyz.scm (emacs-fb2-reader): New variable.
Thank you. Applied, with the changes below.
> +(define-public emacs-fb2-reader
> + (let ((revision "0")
> + (commit "9836db284749e0cef4c43c2cb5358c82ae9b8589")) ; version bump
> + (package
> + (name "emacs-fb2-reader")
> + (version (git-version "0.1.1" revision commit))
No need for `git-version' for version bumps.
> + (native-inputs
> + (list emacs-buttercup
> + emacs-dash
> + emacs-s
> + emacs-async))
> + (propagated-inputs
> + (list emacs-visual-fill-column
> + emacs-f))
I reordered inputs alphabetically.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-06-10 18:47 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-04 16:30 [bug#63900] [PATCH] gnu: Add emacs-fb2-reader Adam Kandur
2023-06-05 16:15 ` Nicolas Goaziou
2023-06-06 20:35 ` Adam
2023-06-10 10:38 ` Adam Kandur
2023-06-10 18:46 ` bug#63900: " Nicolas Goaziou
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).