unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Update URL to dtache.el package
@ 2022-04-27 16:35 Niklas Eklund
  2022-04-27 19:16 ` Tassilo Horn
  2022-04-27 22:06 ` Stefan Monnier
  0 siblings, 2 replies; 24+ messages in thread
From: Niklas Eklund @ 2022-04-27 16:35 UTC (permalink / raw)
  To: emacs-devel

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

Hello,

I have moved the development of dtache.el, which can be found in ELPA,
from gitlab to sourcehut. For reference see this commit:
https://gitlab.com/niklaseklund/dtache/-/commit/b762fe472f1a47d0f1808620d4468aee645654ee

Here is a patch that updates the URL to the new location.

Best regards,
Niklas


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: update-dtache-url --]
[-- Type: text/x-patch, Size: 688 bytes --]

From 8c5f56212b3057c5d311dab1f46be3c20c082a26 Mon Sep 17 00:00:00 2001
From: Niklas Eklund <niklas.eklund@posteo.net>
Date: Wed, 27 Apr 2022 18:27:51 +0200
Subject: [PATCH] * elpa-packages (dtache): Update URL

---
 elpa-packages | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/elpa-packages b/elpa-packages
index 06003bc5fb..606f49deea 100644
--- a/elpa-packages
+++ b/elpa-packages
@@ -187,7 +187,7 @@
  ("dismal"		:url nil)
  ("djvu"		:url nil)
  ("docbook"		:url nil)
- ("dtache"		:url "https://gitlab.com/niklaseklund/dtache"
+ ("dtache"		:url "https://git.sr.ht/~niklaseklund/dtache"
   :news "CHANGELOG.org"
   :readme "README.org"
   :auto-sync t)
-- 
2.34.0


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

* Re: Update URL to dtache.el package
  2022-04-27 16:35 Update URL to dtache.el package Niklas Eklund
@ 2022-04-27 19:16 ` Tassilo Horn
  2022-04-27 22:06 ` Stefan Monnier
  1 sibling, 0 replies; 24+ messages in thread
From: Tassilo Horn @ 2022-04-27 19:16 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: emacs-devel

Niklas Eklund <niklas.eklund@posteo.net> writes:

Hi Niklas,

> I have moved the development of dtache.el, which can be found in ELPA,
> from gitlab to sourcehut. For reference see this commit:
> https://gitlab.com/niklaseklund/dtache/-/commit/b762fe472f1a47d0f1808620d4468aee645654ee
>
> Here is a patch that updates the URL to the new location.

Thanks, pushed.

Bye,
Tassilo



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

* Re: Update URL to dtache.el package
  2022-04-27 16:35 Update URL to dtache.el package Niklas Eklund
  2022-04-27 19:16 ` Tassilo Horn
@ 2022-04-27 22:06 ` Stefan Monnier
  2022-05-01 21:29   ` Niklas Eklund
  1 sibling, 1 reply; 24+ messages in thread
From: Stefan Monnier @ 2022-04-27 22:06 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: emacs-devel

Niklas Eklund [2022-04-27 16:35:00] wrote:
> I have moved the development of dtache.el, which can be found in ELPA,
> from gitlab to sourcehut. For reference see this commit:
> https://gitlab.com/niklaseklund/dtache/-/commit/b762fe472f1a47d0f1808620d4468aee645654ee
> Here is a patch that updates the URL to the new location.

Thanks.  Now we have a problem because the two have diverged:

    % make fetch/dtache
    emacs --batch -l admin/elpa-admin.el -f elpaa-batch-fetch-and-show "dtache"
    Fetching updates for dtache...
    Upstream of dtache has DIVERGED!
    
      Local changes:
    Depuis https://git.sr.ht/~niklaseklund/dtache
     * [nouvelle référence]               -> upstream/dtache/main
    b762fe472f  niklas.eklund@..  Announce development migration
    
      Upstream changes:
    bef9614c76  niklas.eklund@..  Update README with build badge
    7fe012d48e  niklas.eklund@..  Address redraw in tests
    35f8f636a7  niklas.eklund@..  Add .build.yml file
    6b595f4946  niklas.eklund@..  Add markdown README
    
    % 

Could you `git merge b762fe472f` (*not* rebase) into the SourceHut
branch, so that `elpa.git` can continue tracking that new branch?


        Stefan




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

* Re: Update URL to dtache.el package
  2022-04-27 22:06 ` Stefan Monnier
@ 2022-05-01 21:29   ` Niklas Eklund
  2022-05-01 21:36     ` Niklas Eklund
  0 siblings, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-01 21:29 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

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

> Niklas Eklund [2022-04-27 16:35:00] wrote:
>> I have moved the development of dtache.el, which can be found in ELPA,
>> from gitlab to sourcehut. For reference see this commit:
>> https://gitlab.com/niklaseklund/dtache/-/commit/b762fe472f1a47d0f1808620d4468aee645654ee
>> Here is a patch that updates the URL to the new location.
>
> Thanks.  Now we have a problem because the two have diverged:
>
>     % make fetch/dtache
>     emacs --batch -l admin/elpa-admin.el -f elpaa-batch-fetch-and-show "dtache"
>     Fetching updates for dtache...
>     Upstream of dtache has DIVERGED!
>     
>       Local changes:
>     Depuis https://git.sr.ht/~niklaseklund/dtache
>      * [nouvelle référence]               -> upstream/dtache/main
>     b762fe472f  niklas.eklund@..  Announce development migration
>     
>       Upstream changes:
>     bef9614c76  niklas.eklund@..  Update README with build badge
>     7fe012d48e  niklas.eklund@..  Address redraw in tests
>     35f8f636a7  niklas.eklund@..  Add .build.yml file
>     6b595f4946  niklas.eklund@..  Add markdown README
>     
>     % 
>
> Could you `git merge b762fe472f` (*not* rebase) into the SourceHut
> branch, so that `elpa.git` can continue tracking that new branch?
>
>
>         Stefan

Hello Stefan,

I have seemed to missed your mail. Was just thinking about mailing you
and asking what the issue might be since elpa didn't seem to catch on
to the new URL.

I'll try your suggestion.

/Niklas



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

* Re: Update URL to dtache.el package
  2022-05-01 21:29   ` Niklas Eklund
@ 2022-05-01 21:36     ` Niklas Eklund
  2022-05-02  6:18       ` Stefan Monnier
  0 siblings, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-01 21:36 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

Niklas Eklund <niklas.eklund@posteo.net> writes:

> Hello Stefan,
>
> I have seemed to missed your mail. Was just thinking about mailing you
> and asking what the issue might be since elpa didn't seem to catch on
> to the new URL.
>
> I'll try your suggestion.

I have run the merge command and pushed it to the main branch. Could you
verify if that looks correct now?

/Niklas



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

* Re: Update URL to dtache.el package
  2022-05-01 21:36     ` Niklas Eklund
@ 2022-05-02  6:18       ` Stefan Monnier
  2022-05-08 23:36         ` Name dtache.el Richard Stallman
  0 siblings, 1 reply; 24+ messages in thread
From: Stefan Monnier @ 2022-05-02  6:18 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: emacs-devel

Niklas Eklund [2022-05-01 21:36:44] wrote:
> Niklas Eklund <niklas.eklund@posteo.net> writes:
>> Hello Stefan,
>> I have seemed to missed your mail. Was just thinking about mailing you
>> and asking what the issue might be since elpa didn't seem to catch on
>> to the new URL.
>>
>> I'll try your suggestion.
>
> I have run the merge command and pushed it to the main branch. Could you
> verify if that looks correct now?

Looks good, thanks,


        Stefan




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

* Name dtache.el
  2022-05-02  6:18       ` Stefan Monnier
@ 2022-05-08 23:36         ` Richard Stallman
  2022-05-09 19:26           ` Niklas Eklund
  0 siblings, 1 reply; 24+ messages in thread
From: Richard Stallman @ 2022-05-08 23:36 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: niklas.eklund, emacs-devel

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

The name "dtache" is less helpful than it could be.  How about
renaming it to the clearest name we can think of?  "detached" would be
an improvement, but perhaps we can find one even better.

-- 
Dr Richard Stallman (https://stallman.org)
Chief GNUisance of the GNU Project (https://gnu.org)
Founder, Free Software Foundation (https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)





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

* Re: Name dtache.el
  2022-05-08 23:36         ` Name dtache.el Richard Stallman
@ 2022-05-09 19:26           ` Niklas Eklund
  2022-05-20 14:23             ` Niklas Eklund
  0 siblings, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-09 19:26 UTC (permalink / raw)
  To: rms, Stefan Monnier; +Cc: emacs-devel

Richard Stallman <rms@gnu.org> writes:

> [[[ To any NSA and FBI agents reading my email: please consider    ]]]
> [[[ whether defending the US Constitution against all enemies,     ]]]
> [[[ foreign or domestic, requires you to follow Snowden's example. ]]]
>
> The name "dtache" is less helpful than it could be.  How about
> renaming it to the clearest name we can think of?  "detached" would be
> an improvement, but perhaps we can find one even better.

Its a good point, thanks for the input. I think its current name makes
sense once you know about the package, but it might not be the best in
giving others a sense of its capabilities.

In short I think what the package provides can be summarized as:
- the ability to launch detached processes
- a convenient interface to interact with said processes, or integrate
them into Emacs

I agree that the word "detached" is more clear than its current name. An
additional benefit with that name, which I didn't think about first,
would be that it ends with ed. That would make it align better with
other tools such as dired/proced which provides functionalities that
resembles the second bullet in my list above.

/Niklas

> -- 
> Dr Richard Stallman (https://stallman.org)
> Chief GNUisance of the GNU Project (https://gnu.org)
> Founder, Free Software Foundation (https://fsf.org)
> Internet Hall-of-Famer (https://internethalloffame.org)



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

* Re: Name dtache.el
  2022-05-09 19:26           ` Niklas Eklund
@ 2022-05-20 14:23             ` Niklas Eklund
  2022-05-20 14:31               ` Stefan Monnier
  0 siblings, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-20 14:23 UTC (permalink / raw)
  To: rms, Stefan Monnier; +Cc: emacs-devel

Niklas Eklund <niklas.eklund@posteo.net> writes:

Hi,

I have gotten around to update the name of the package from dtache ->
detached.el. Here is a patch that can be applied to the elpa repository
in order to update the elpa-packages file.

I am not sure what to do about the name of the branch, if I was suppose
to update that or if a maintainer can do that to make it aligned with
the new name.

/Niklas

> Richard Stallman <rms@gnu.org> writes:
>
>> [[[ To any NSA and FBI agents reading my email: please consider    ]]]
>> [[[ whether defending the US Constitution against all enemies,     ]]]
>> [[[ foreign or domestic, requires you to follow Snowden's example. ]]]
>>
>> The name "dtache" is less helpful than it could be.  How about
>> renaming it to the clearest name we can think of?  "detached" would be
>> an improvement, but perhaps we can find one even better.
>
> Its a good point, thanks for the input. I think its current name makes
> sense once you know about the package, but it might not be the best in
> giving others a sense of its capabilities.
>
> In short I think what the package provides can be summarized as:
> - the ability to launch detached processes
> - a convenient interface to interact with said processes, or integrate
> them into Emacs
>
> I agree that the word "detached" is more clear than its current name. An
> additional benefit with that name, which I didn't think about first,
> would be that it ends with ed. That would make it align better with
> other tools such as dired/proced which provides functionalities that
> resembles the second bullet in my list above.
>
> /Niklas
>
>> -- 
>> Dr Richard Stallman (https://stallman.org)
>> Chief GNUisance of the GNU Project (https://gnu.org)
>> Founder, Free Software Foundation (https://fsf.org)
>> Internet Hall-of-Famer (https://internethalloffame.org)



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

* Re: Name dtache.el
  2022-05-20 14:23             ` Niklas Eklund
@ 2022-05-20 14:31               ` Stefan Monnier
  2022-05-20 15:19                 ` Niklas Eklund
  2022-05-20 15:50                 ` Eli Zaretskii
  0 siblings, 2 replies; 24+ messages in thread
From: Stefan Monnier @ 2022-05-20 14:31 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: rms, emacs-devel

Niklas Eklund [2022-05-20 14:23:46] wrote:
> I have gotten around to update the name of the package from dtache ->
> detached.el.

Is it really necessary?

> I am not sure what to do about the name of the branch, if I was suppose
> to update that or if a maintainer can do that to make it aligned with
> the new name.

Renaming requires some manual tweaking, yes.  I can take care of that.
But more importantly it's an annoyance for your users since updates
won't be applied for them, so they may end up stuck at the old
`dtache.el` release without realizing there's a new version under
a different name.  We currently don't have any way to inform
`package-update` and friends about a renaming :-(


        Stefan




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

* Re: Name dtache.el
  2022-05-20 14:31               ` Stefan Monnier
@ 2022-05-20 15:19                 ` Niklas Eklund
  2022-05-20 15:25                   ` Stefan Monnier
  2022-05-20 15:50                 ` Eli Zaretskii
  1 sibling, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-20 15:19 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: rms, emacs-devel


May 20, 2022 16:32:05 Stefan Monnier <monnier@iro.umontreal.ca>:

> Niklas Eklund [2022-05-20 14:23:46] wrote:
>> I have gotten around to update the name of the package from dtache ->
>> detached.el.
>
> Is it really necessary?

I think it's desirable to make the switch, even though I dislike the 
annoyance it creates. I think its good to do the switch now then later 
on.

I have done commits renaming all the files and functions so updating the 
package archives is the last step.

>> I am not sure what to do about the name of the branch, if I was 
>> suppose
>> to update that or if a maintainer can do that to make it aligned with
>> the new name.
>
> Renaming requires some manual tweaking, yes.  I can take care of that.

Great

> But more importantly it's an annoyance for your users since updates
> won't be applied for them, so they may end up stuck at the old
> `dtache.el` release without realizing there's a new version under
> a different name.  We currently don't have any way to inform
> `package-update` and friends about a renaming :-(

Yes, that's unfortunate of course. Since I merged the renames of files 
and functions to the new names I thought that would indirectly inform 
users about the change.

Although those relying on releases won't get that information then. Maybe 
one solution to that is that I release a new version and only update the 
recipe ones it has become available on ELPA?

/Niklas



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

* Re: Name dtache.el
  2022-05-20 15:19                 ` Niklas Eklund
@ 2022-05-20 15:25                   ` Stefan Monnier
  2022-05-20 15:36                     ` Niklas Eklund
  0 siblings, 1 reply; 24+ messages in thread
From: Stefan Monnier @ 2022-05-20 15:25 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: rms, emacs-devel

> Although those relying on releases won't get that information then. Maybe
>  one solution to that is that I release a new version and only update the
>  recipe ones it has become available on ELPA?

A partial solution is to make a release of `detached.el` and once that's
done, make a release of `dtache.el` which `Package-Requires: ((detached "??"))`
It can made of a mostly empty file or a file containing just compatibility aliases.


        Stefan




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

* Re: Name dtache.el
  2022-05-20 15:25                   ` Stefan Monnier
@ 2022-05-20 15:36                     ` Niklas Eklund
  2022-05-20 16:33                       ` Niklas Eklund
  2022-05-20 18:58                       ` Stefan Monnier
  0 siblings, 2 replies; 24+ messages in thread
From: Niklas Eklund @ 2022-05-20 15:36 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: rms, emacs-devel


May 20, 2022 17:25:24 Stefan Monnier <monnier@iro.umontreal.ca>:

>> Although those relying on releases won't get that information then. 
>> Maybe
>> one solution to that is that I release a new version and only update 
>> the
>> recipe ones it has become available on ELPA?
>
> A partial solution is to make a release of `detached.el` and once 
> that's
> done, make a release of `dtache.el` which `Package-Requires: ((detached 
> "??"))`

Would I then need to have two different git repositories or how would you 
go about doing that?

> It can made of a mostly empty file or a file containing just 
> compatibility aliases.

So would we then keep both dtache and detached in the elpa-packages file 
or how would that work?

Another option I could think about is making a release of the package at 
its current path https://git.sr.ht/~niklaseklund/dtache which will be its 
last dtache release. But just before the release I add a commit that 
issues a warning that if you are running this release you are tracking 
the wrong patch.

Then after the release, I remove that warning. Rename the https path to 
detached.el and make a release with the new package name.

Then it should be fine to update elpas tracking. Because if they run on 
the old release the should have the warning that they need to upgrade. 
What do you think?

/Niklas


but embedding a warning when it loads that




>
>         Stefan



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

* Re: Name dtache.el
  2022-05-20 14:31               ` Stefan Monnier
  2022-05-20 15:19                 ` Niklas Eklund
@ 2022-05-20 15:50                 ` Eli Zaretskii
  1 sibling, 0 replies; 24+ messages in thread
From: Eli Zaretskii @ 2022-05-20 15:50 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: niklas.eklund, rms, emacs-devel

> From: Stefan Monnier <monnier@iro.umontreal.ca>
> Cc: rms@gnu.org,  emacs-devel@gnu.org
> Date: Fri, 20 May 2022 10:31:52 -0400
> 
> Niklas Eklund [2022-05-20 14:23:46] wrote:
> > I have gotten around to update the name of the package from dtache ->
> > detached.el.
> 
> Is it really necessary?

Maybe it isn't, but the new name is definitely better, IMO.



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

* Re: Name dtache.el
  2022-05-20 15:36                     ` Niklas Eklund
@ 2022-05-20 16:33                       ` Niklas Eklund
  2022-05-20 18:58                       ` Stefan Monnier
  1 sibling, 0 replies; 24+ messages in thread
From: Niklas Eklund @ 2022-05-20 16:33 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: rms, emacs-devel

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

Niklas Eklund <niklas.eklund@posteo.net> writes:

> Another option I could think about is making a release of the package at 
> its current path https://git.sr.ht/~niklaseklund/dtache which will be its 
> last dtache release. But just before the release I add a commit that 
> issues a warning that if you are running this release you are tracking 
> the wrong patch.

Here is a patch of what I meant. Applying this prior to the 0.7 release so
that it is included. Then removing it and making a 0.8 which will be
found on the new https location.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-patch, Size: 2824 bytes --]

From c8e43b32807e4c0aa396d989bd0146b764aa8973 Mon Sep 17 00:00:00 2001
From: Niklas Eklund <niklas.eklund@posteo.net>
Date: Fri, 20 May 2022 18:10:38 +0200
Subject: [PATCH] Warn users to update

---
 detached.el |  9 +++++++++
 dtache.el   | 37 +++++++++++++++++++++++++++++++++++++
 2 files changed, 46 insertions(+)
 create mode 100644 dtache.el

diff --git a/detached.el b/detached.el
index f5a0ac9..5d2fea3 100644
--- a/detached.el
+++ b/detached.el
@@ -53,6 +53,15 @@
 
 (declare-function detached-eshell-get-dtach-process "detached-eshell")
 
+(warn
+ "The `dtach' package has been renamed to `detached'.
+
+If you are using ELPA to install the package make sure to update your configuration to install detached instead of dtache.
+If you are installing directly from source, make sure its https://git.sr.ht/~niklaseklund/detached.el
+
+The version of detached should be 0.8 or newer, if its not you are tracking the wrong source, after a successful update this message should not be present anymore.
+Remember to follow the README for instructions on how to configure `detached.el.")
+
 ;;;; Variables
 
 ;;;;; Customizable
diff --git a/dtache.el b/dtache.el
new file mode 100644
index 0000000..9956468
--- /dev/null
+++ b/dtache.el
@@ -0,0 +1,37 @@
+;;; dtache.el --- A package to launch, and manage, detached processes -*- lexical-binding: t -*-
+
+;; Copyright (C) 2020-2022  Free Software Foundation, Inc.
+
+;; This file is part of GNU Emacs.
+
+;; This program is free software; you can redistribute it and/or modify
+;; it under the terms of the GNU General Public License as published by
+;; the Free Software Foundation, either version 3 of the License, or
+;; (at your option) any later version.
+
+;; This program is distributed in the hope that it will be useful,
+;; but WITHOUT ANY WARRANTY; without even the implied warranty of
+;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;; GNU General Public License for more details.
+
+;; You should have received a copy of the GNU General Public License
+;; along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+;;; Commentary:
+
+;; This is `dtache' the old package file you should switch to `detached'.
+
+;;; Code:
+
+(warn
+ "The `dtach' package has been renamed to `detached'.
+
+If you are using ELPA to install the package make sure to update your configuration to install detached instead of dtache.
+If you are installing directly from source, make sure its https://git.sr.ht/~niklaseklund/detached.el
+
+The version of detached should be 0.8 or newer, if its not you are tracking the wrong source, after a successful update this message should not be present anymore.
+Remember to follow the README for instructions on how to configure `detached.el.")
+
+(provide 'dtache)
+
+;;; dtache.el ends here
-- 
2.34.0


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


/Niklas



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

* Re: Name dtache.el
  2022-05-20 15:36                     ` Niklas Eklund
  2022-05-20 16:33                       ` Niklas Eklund
@ 2022-05-20 18:58                       ` Stefan Monnier
  2022-05-20 19:27                         ` Niklas Eklund
  1 sibling, 1 reply; 24+ messages in thread
From: Stefan Monnier @ 2022-05-20 18:58 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: rms, emacs-devel

> Would I then need to have two different git repositories or how would you go
> about doing that?

Yes.  Would that be a problem?


        Stefan




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

* Re: Name dtache.el
  2022-05-20 18:58                       ` Stefan Monnier
@ 2022-05-20 19:27                         ` Niklas Eklund
  2022-05-20 20:44                           ` Niklas Eklund
  0 siblings, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-20 19:27 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: rms, emacs-devel

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

>> Would I then need to have two different git repositories or how would you go
>> about doing that?
>
> Yes.  Would that be a problem?

No, not really, I just asked to see if I understood what you meant :)

So your proposal would be to:
1) Create a new git repository for detached.el
2) Add this package to ELPA
3) Release a new version detached.el
4) Update the original dtache package to require detached
5) Make a new release of dtache

Is this what you meant? A question about adding a require in dtache, do
I need to specify this dependency in elpa somewhere for it to work properly?

/Niklas



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

* Re: Name dtache.el
  2022-05-20 19:27                         ` Niklas Eklund
@ 2022-05-20 20:44                           ` Niklas Eklund
  2022-05-20 21:00                             ` Stefan Monnier
  0 siblings, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-20 20:44 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: rms, emacs-devel

Niklas Eklund <niklas.eklund@posteo.net> writes:

> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>
>>> Would I then need to have two different git repositories or how would you go
>>> about doing that?
>>
>> Yes.  Would that be a problem?
>
> No, not really, I just asked to see if I understood what you meant :)
>
> So your proposal would be to:
> 1) Create a new git repository for detached.el
> 2) Add this package to ELPA
> 3) Release a new version detached.el
> 4) Update the original dtache package to require detached
> 5) Make a new release of dtache
>
> Is this what you meant? A question about adding a require in dtache, do
> I need to specify this dependency in elpa somewhere for it to work properly?

Ah ok you meant adding a line similar to this one in dtache.el?

;; Package-Requires: ((company "0.8.0") (math-symbol-lists "1.3"))

Seems like a smart way to communicate to users that they need to install detached.el

> /Niklas



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

* Re: Name dtache.el
  2022-05-20 20:44                           ` Niklas Eklund
@ 2022-05-20 21:00                             ` Stefan Monnier
  2022-05-20 21:21                               ` Niklas Eklund
  0 siblings, 1 reply; 24+ messages in thread
From: Stefan Monnier @ 2022-05-20 21:00 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: rms, emacs-devel

>> So your proposal would be to:
>> 1) Create a new git repository for detached.el
>> 2) Add this package to ELPA
>> 3) Release a new version detached.el
>> 4) Update the original dtache package to require detached
>> 5) Make a new release of dtache

Yup.

> Ah ok you meant adding a line similar to this one in dtache.el?
>
> ;; Package-Requires: ((company "0.8.0") (math-symbol-lists "1.3"))

Yes.

> Seems like a smart way to communicate to users that they need to install detached.el

This way they'll "automatically" be moved to `detached.el`.

It's not quite right because `detached.el` will be considered as
"automatically added" so `package-autoremove` might do the wrong thing,
but I think it's currently as close as we can get to the ideal.


        Stefan




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

* Re: Name dtache.el
  2022-05-20 21:00                             ` Stefan Monnier
@ 2022-05-20 21:21                               ` Niklas Eklund
  2022-05-20 21:35                                 ` Stefan Monnier
  0 siblings, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-20 21:21 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: rms, emacs-devel

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

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

>>> So your proposal would be to:
>>> 1) Create a new git repository for detached.el
>>> 2) Add this package to ELPA
>>> 3) Release a new version detached.el
>>> 4) Update the original dtache package to require detached
>>> 5) Make a new release of dtache
>
> Yup.

Great

>> Ah ok you meant adding a line similar to this one in dtache.el?
>>
>> ;; Package-Requires: ((company "0.8.0") (math-symbol-lists "1.3"))
>
> Yes.
>
Roger that!

>> Seems like a smart way to communicate to users that they need to install detached.el
>
> This way they'll "automatically" be moved to `detached.el`.

Sounds great.

> It's not quite right because `detached.el` will be considered as
> "automatically added" so `package-autoremove` might do the wrong thing,
> but I think it's currently as close as we can get to the ideal.

Alright, let's hope for the best then :)

Here is a patch adding detached.el to the list of packages.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-patch, Size: 996 bytes --]

From 084f3c644ce90767173c22fa49d07feaf5c1ca09 Mon Sep 17 00:00:00 2001
From: Niklas Eklund <niklas.eklund@posteo.net>
Date: Fri, 20 May 2022 23:16:22 +0200
Subject: [PATCH] * elpa-packages (dtache): Add detached.el

The dtache package has been renamed to detached. This patch adds the
renamed package to the list of elpa packages.

The dtache package will remain for now and detached will be added as a
required package to it.
---
 elpa-packages | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/elpa-packages b/elpa-packages
index ac322f4cc4..6bfd4d7cfa 100644
--- a/elpa-packages
+++ b/elpa-packages
@@ -175,6 +175,9 @@
   :auto-sync t)
  ("devdocs"		:url "https://github.com/astoff/devdocs.el"
   :auto-sync t)
+ ("detached"		:url "https://git.sr.ht/~niklaseklund/detached.el"
+  :news "CHANGELOG.org"
+  :auto-sync t)
  ("dict-tree"		:url "http://www.dr-qubit.org/git/predictive.git")
  ("diffview"            :url "https://github.com/mgalgs/diffview-mode.git")
  ("diff-hl"
-- 
2.34.0


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



/Niklas

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

* Re: Name dtache.el
  2022-05-20 21:21                               ` Niklas Eklund
@ 2022-05-20 21:35                                 ` Stefan Monnier
  2022-05-21  7:38                                   ` Niklas Eklund
  0 siblings, 1 reply; 24+ messages in thread
From: Stefan Monnier @ 2022-05-20 21:35 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: rms, emacs-devel

> Here is a patch adding detached.el to the list of packages.

Thanks, pushed (with a `:readme "README.md"`).


        Stefan




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

* Re: Name dtache.el
  2022-05-20 21:35                                 ` Stefan Monnier
@ 2022-05-21  7:38                                   ` Niklas Eklund
  2022-05-21 14:44                                     ` Stefan Monnier
  0 siblings, 1 reply; 24+ messages in thread
From: Niklas Eklund @ 2022-05-21  7:38 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: rms, emacs-devel

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

>> Here is a patch adding detached.el to the list of packages.
>
> Thanks, pushed (with a `:readme "README.md"`).
>
>
>         Stefan

Super, thanks! :)

I can see that a branch has been created at ELPA and that it seems to
track it correctly with the commits.

But I haven't seen it show up in the list of packages at
http://elpa.gnu.org/devel/. In this situations is there something I can
do to debug and understand why it doesn't show up?

/Niklas



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

* Re: Name dtache.el
  2022-05-21  7:38                                   ` Niklas Eklund
@ 2022-05-21 14:44                                     ` Stefan Monnier
  2022-05-21 16:13                                       ` Niklas Eklund
  0 siblings, 1 reply; 24+ messages in thread
From: Stefan Monnier @ 2022-05-21 14:44 UTC (permalink / raw)
  To: Niklas Eklund; +Cc: rms, emacs-devel

> But I haven't seen it show up in the list of packages at
> http://elpa.gnu.org/devel/. In this situations is there something I can
> do to debug and understand why it doesn't show up?

Other than make time go faster, you mean?  :-)
[ both GNU and GNU-devel ELPAs are updated once every ~12h.  ]


        Stefan




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

* Re: Name dtache.el
  2022-05-21 14:44                                     ` Stefan Monnier
@ 2022-05-21 16:13                                       ` Niklas Eklund
  0 siblings, 0 replies; 24+ messages in thread
From: Niklas Eklund @ 2022-05-21 16:13 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: rms, emacs-devel


May 21, 2022 16:44:38 Stefan Monnier <monnier@iro.umontreal.ca>:

>> But I haven't seen it show up in the list of packages at
>> http://elpa.gnu.org/devel/. In this situations is there something I 
>> can
>> do to debug and understand why it doesn't show up?
>
> Other than make time go faster, you mean?  :-)

Haha, yes I am missing that feature ;)

> [ both GNU and GNU-devel ELPAs are updated once every ~12h.  ]

Thanks for the clarification, it has indeed showed up now! :)

/Niklas
>
>         Stefan



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

end of thread, other threads:[~2022-05-21 16:13 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-27 16:35 Update URL to dtache.el package Niklas Eklund
2022-04-27 19:16 ` Tassilo Horn
2022-04-27 22:06 ` Stefan Monnier
2022-05-01 21:29   ` Niklas Eklund
2022-05-01 21:36     ` Niklas Eklund
2022-05-02  6:18       ` Stefan Monnier
2022-05-08 23:36         ` Name dtache.el Richard Stallman
2022-05-09 19:26           ` Niklas Eklund
2022-05-20 14:23             ` Niklas Eklund
2022-05-20 14:31               ` Stefan Monnier
2022-05-20 15:19                 ` Niklas Eklund
2022-05-20 15:25                   ` Stefan Monnier
2022-05-20 15:36                     ` Niklas Eklund
2022-05-20 16:33                       ` Niklas Eklund
2022-05-20 18:58                       ` Stefan Monnier
2022-05-20 19:27                         ` Niklas Eklund
2022-05-20 20:44                           ` Niklas Eklund
2022-05-20 21:00                             ` Stefan Monnier
2022-05-20 21:21                               ` Niklas Eklund
2022-05-20 21:35                                 ` Stefan Monnier
2022-05-21  7:38                                   ` Niklas Eklund
2022-05-21 14:44                                     ` Stefan Monnier
2022-05-21 16:13                                       ` Niklas Eklund
2022-05-20 15:50                 ` Eli Zaretskii

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

	https://git.savannah.gnu.org/cgit/emacs.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).