[ELPA] New package: taxy-magit-section

[ELPA] New package: taxy-magit-section

[Adam Porter]

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

Hi Stefan, et al,

Having developed it a bit, I'd like to split taxy-magit-section into a
separate package while keeping it in the same git repo.  Please see the
attached patch to elpa-packages; I hope I did it correctly.

-- 
Thanks,
Adam

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-elpa-packages-taxy-taxy-magit-section-Update-add.patch --]
[-- Type: text/x-diff, Size: 1003 bytes --]

From 1675e898380003c6ffdb10318ab700e6b51b0427 Mon Sep 17 00:00:00 2001
From: Adam Porter <adam@alphapapa.net>
Date: Fri, 24 Sep 2021 05:34:37 +0000
Subject: [PATCH] * elpa-packages (taxy, taxy-magit-section): Update/add

---
 elpa-packages | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/elpa-packages b/elpa-packages
index cffd9a63ca..514ed04fd0 100644
--- a/elpa-packages
+++ b/elpa-packages
@@ -373,6 +373,11 @@
  ("system-packages"	:url "https://gitlab.com/jabranham/system-packages")
  ("systemd"		:url nil)
  ("taxy"                :url "https://github.com/alphapapa/taxy.el.git"
+  :doc "taxy.info"
+  :ignored-files ("taxy-magit-section.el" "images")
+  :auto-sync t)
+ ("taxy-magit-section"  :url "https://github.com/alphapapa/taxy.el.git"
+  :ignored-files ("taxy.el" "examples" "images" "taxy.info")
   :auto-sync t)
  ("temp-buffer-browse"  :url "https://github.com/leoliu/temp-buffer-browse")
  ("test-simple"         :url "https://github.com/rocky/emacs-test-simple")
-- 
2.20.1

Re: [ELPA] New package: taxy-magit-section

[Stefan Monnier]

> Having developed it a bit, I'd like to split taxy-magit-section into a
> separate package while keeping it in the same git repo.  Please see the
> attached patch to elpa-packages; I hope I did it correctly.

It looks correct in the sense that it should work and give the right
result, but it's *much* better to try and avoid such situations by
splitting it into a separate repository/branch: releasing several ELPA
packages from the same repository introduces various problems (none of
them really serious, luckily, so it will work but it's better avoided).


        Stefan

Re: [ELPA] New package: taxy-magit-section

[Adam Porter]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> Having developed it a bit, I'd like to split taxy-magit-section into a
>> separate package while keeping it in the same git repo.  Please see the
>> attached patch to elpa-packages; I hope I did it correctly.
>
> It looks correct in the sense that it should work and give the right
> result, but it's *much* better to try and avoid such situations by
> splitting it into a separate repository/branch: releasing several ELPA
> packages from the same repository introduces various problems (none of
> them really serious, luckily, so it will work but it's better avoided).

Okay, I'll see about putting it in a separate branch.  Thanks.

Re: [ELPA] New package: taxy-magit-section

[Adam Porter]

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

Hi Stefan,

> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
>>> Having developed it a bit, I'd like to split taxy-magit-section into a
>>> separate package while keeping it in the same git repo.  Please see the
>>> attached patch to elpa-packages; I hope I did it correctly.
>>
>> It looks correct in the sense that it should work and give the right
>> result, but it's *much* better to try and avoid such situations by
>> splitting it into a separate repository/branch: releasing several ELPA
>> packages from the same repository introduces various problems (none of
>> them really serious, luckily, so it will work but it's better avoided).

I've moved taxy-magit-section.el to a separate branch in the taxy.el
repo.  Here's an updated patch:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-elpa-packages-taxy-taxy-magit-section-Update-add.patch --]
[-- Type: text/x-diff, Size: 985 bytes --]

From e13670366f679b5a1af82c605f67a0fc46d9e90f Mon Sep 17 00:00:00 2001
From: Adam Porter <adam@alphapapa.net>
Date: Fri, 24 Sep 2021 05:34:37 +0000
Subject: [PATCH] * elpa-packages (taxy, taxy-magit-section): Update/add

---
 elpa-packages | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/elpa-packages b/elpa-packages
index 96b6f55..0bdcb69 100644
--- a/elpa-packages
+++ b/elpa-packages
@@ -385,6 +385,12 @@
  ("system-packages"	:url "https://gitlab.com/jabranham/system-packages")
  ("systemd"		:url nil)
  ("taxy"                :url "https://github.com/alphapapa/taxy.el.git"
+  :doc "taxy.info"
+  :ignored-files ("images")
+  :auto-sync t)
+ ("taxy-magit-section"  :url "https://github.com/alphapapa/taxy.el.git"
+  :branch "package/taxy-magit-section"
+  :doc "taxy-magit-section.info"
   :auto-sync t)
  ("temp-buffer-browse"  :url "https://github.com/leoliu/temp-buffer-browse")
  ("test-simple"         :url "https://github.com/rocky/emacs-test-simple")
-- 
2.7.4


[-- Attachment #3: Type: text/plain, Size: 17 bytes --]


--
Thanks,
Adam

Re: [ELPA] New package: taxy-magit-section

[Stefan Monnier]

> diff --git a/elpa-packages b/elpa-packages
> index 96b6f55..0bdcb69 100644
> --- a/elpa-packages
> +++ b/elpa-packages
> @@ -385,6 +385,12 @@
>   ("system-packages"	:url "https://gitlab.com/jabranham/system-packages")
>   ("systemd"		:url nil)
>   ("taxy"                :url "https://github.com/alphapapa/taxy.el.git"
> +  :doc "taxy.info"
> +  :ignored-files ("images")
> +  :auto-sync t)
> + ("taxy-magit-section"  :url "https://github.com/alphapapa/taxy.el.git"
> +  :branch "package/taxy-magit-section"
> +  :doc "taxy-magit-section.info"
>    :auto-sync t)
>   ("temp-buffer-browse"  :url "https://github.com/leoliu/temp-buffer-browse")
>   ("test-simple"         :url "https://github.com/rocky/emacs-test-simple")

Thanks, installed.

But please note that it would be much better not to store the `.info`
files in the repository (the general rule is that you only want to
store *source* files in there) and instead use

    :doc "README.org"


-- Stefan

Re: [ELPA] New package: taxy-magit-section

[Adam Porter]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> Thanks, installed.

Thanks.

> But please note that it would be much better not to store the `.info`
> files in the repository (the general rule is that you only want to
> store *source* files in there) and instead use
>
>     :doc "README.org"

I'd generally be glad to do that.  I did it myself when the README.org
file is saved because that's necessary for my packages that are on
MELPA.  I wouldn't mind for ELPA to do that automatically, however there
is one issue: users who install the package by other means, like with
Quelpa (as I do for some of my own packages, so I don't have to wait for
(M)ELPA to build them) or Straight wouldn't get the info manual, IIUC.

Also, it can be helpful for me to see the diffs in the generated info
manual when I'm committing changes to the readme.

So, for now at least, I'd prefer to keep them in the repo and generate
them myself, if that's acceptable.

Re: [ELPA] New package: taxy-magit-section

[Stefan Monnier]

> I'd generally be glad to do that.  I did it myself when the README.org
> file is saved because that's necessary for my packages that are on
> MELPA.

I'm not sure what's the benefit of having packages on MELPA when they're
already in GNU ELPA, to be honest.

> I wouldn't mind for ELPA to do that automatically, however there
> is one issue: users who install the package by other means, like with
> Quelpa (as I do for some of my own packages, so I don't have to wait for
> (M)ELPA to build them) or Straight wouldn't get the info manual, IIUC.

They'd need to generate it, just like `elpa-admin.el` would do, indeed.

> So, for now at least, I'd prefer to keep them in the repo and generate
> them myself, if that's acceptable.

It's fully up to you.


        Stefan

Re: [ELPA] New package: taxy-magit-section

[Adam Porter]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> I'd generally be glad to do that.  I did it myself when the README.org
>> file is saved because that's necessary for my packages that are on
>> MELPA.
>
> I'm not sure what's the benefit of having packages on MELPA when they're
> already in GNU ELPA, to be honest.

To be clear, I didn't mean that this one is on MELPA and GNU ELPA.  I
meant that I copied that pattern from other packages of mine that are on
MELPA.  :)

Re: [ELPA] New package: taxy-magit-section

[dick]

SM> I'm not sure what's the benefit of having packages on MELPA when they're
SM> already in GNU ELPA, to be honest.

Wow, shots fired!

The answer of course is the same reason why a Target, Walmart, or Amazon
eventually sells eggs and milk on top of whatever knickknacks they originally
sold.  One-stop shopping is a user tropism.  That's why emacs clings to relevance.