* updating async on elpa @ 2021-04-30 11:56 Thierry Volpiatto 2021-04-30 14:17 ` Stefan Monnier 0 siblings, 1 reply; 23+ messages in thread From: Thierry Volpiatto @ 2021-04-30 11:56 UTC (permalink / raw) To: emacs-devel [-- Attachment #1: Type: text/plain, Size: 223 bytes --] Hello, I just pushed last emacs-async changes to Elpa, but I am unsure if I did it correctly as the way to merge externals repos with Elpa have changed, can you verify if all went well. Thanks. -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 11:56 updating async on elpa Thierry Volpiatto @ 2021-04-30 14:17 ` Stefan Monnier 2021-04-30 15:20 ` Thierry Volpiatto 0 siblings, 1 reply; 23+ messages in thread From: Stefan Monnier @ 2021-04-30 14:17 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: emacs-devel > I just pushed last emacs-async changes to Elpa, but I am unsure if I did > it correctly as the way to merge externals repos with Elpa have changed, > can you verify if all went well. Hmm... you pushed a new branch called `externals/emacs-async` and another called `elpa` but you didn't change the branch `externals/async` which is the one that matters ;-) I just tried to sync the `async` code on elpa.git and it tells me: % (cd ../elpa; make sync/async) emacs --batch -l admin/elpa-admin.el -f elpaa-batch-fetch-and-push "async" Fetching updates for async... Upstream of async has DIVERGED! Local changes: Depuis https://github.com/jwiegley/emacs-async d7e7f79ee4..943a75c6a5 -> upstream/async/main 60118ce687 monnier@iro.um.. Get async-pkg.el out of the way d854a60cfa monnier@iro.um.. * async-bytecomp.el (async-bytecomp--get-package-deps): Fix last change a4a50337e4 monnier@iro.um.. Use cl-lib an nadvice (re-apply 783af291680d) Upstream changes: 943a75c6a5 thievol@posteo.. Fix paren error in dired-async d7e7f79ee4 thievol@posteo.. Update README 630708b150 thievol@posteo.. Merge branch 'master' of github.com:jwiegley/emacs-async 4f78c0c91f thievol@posteo.. Merge pull request #138 from Stebalien/fix/byte-comp-prefix 0fd5d54073 steven@stebali.. Make the async byte compile variable injection regexp "shy" 00c6f3f85b steven@stebali.. fix inclusion of bytecomp variables 06f0ec7d96 thievol@posteo.. Fix typo in previous commit 14f48de586 thievol@posteo.. Fix Makefile 36a10151e7 thievol@posteo.. Change default value of async-bytecomp-allowed-packages to 'all 86aef2c38e johnw@newartis.. Untabify and indent-buffer (#119) a1bd2ac49c conao3@gmail.com untabify and indent-buffer 036b475a29 thierry.volpia.. Merge pull request #118 from conao3/fix-warnings fbbba03e98 conao3@gmail.com fix package-lint/check-doc/byte-compiler warnings 11769c1895 thierry.volpia.. Update version in source file as well d255820c5a thierry.volpia.. Merge branch 'master' of github.com:jwiegley/emacs-async e81c685741 thierry.volpia.. Update *pkg.el file 03667dde47 johnw@newartis.. Merge pull request #116 from DamienCassou/typos 64bb180201 damien@cassou.me Fix typos 1209cc2ccd thierry.volpia.. Merge pull request #115 from M4rkD/master c80976d3f5 mark.dawson@sw.. Some rewording of part of README.md 35ab78afb9 thierry.volpia.. Merge pull request #114 from tarsius/forge 6577ae5cf9 jonas@bernoul.li Add forge to async-bytecomp-allowed-packages 67c369555d thierry.volpia.. Fix async-bytecomp--get-package-deps according to % so I think before pushing to `externals/async` someone will have to merge the `elpa.git` changes into https://github.com/jwiegley/emacs-async. Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 14:17 ` Stefan Monnier @ 2021-04-30 15:20 ` Thierry Volpiatto 2021-04-30 16:21 ` Stefan Monnier 0 siblings, 1 reply; 23+ messages in thread From: Thierry Volpiatto @ 2021-04-30 15:20 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel [-- Attachment #1: Type: text/plain, Size: 855 bytes --] Stefan Monnier <monnier@iro.umontreal.ca> writes: > [[PGP Encrypted Part:OK]] >> I just pushed last emacs-async changes to Elpa, but I am unsure if I did >> it correctly as the way to merge externals repos with Elpa have changed, >> can you verify if all went well. > > Hmm... you pushed a new branch called `externals/emacs-async` and > another called `elpa` but you didn't change the branch `externals/async` > which is the one that matters ;-) Not sure how I did this (the second one) :-) sorry. > so I think before pushing to `externals/async` someone will have to > merge the `elpa.git` changes into > https://github.com/jwiegley/emacs-async. I will try to do it, if I can get async from elpa.git, don't know how to get this for now. Thanks. > > Stefan > > > [[End of PGP Encrypted Part]] -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 15:20 ` Thierry Volpiatto @ 2021-04-30 16:21 ` Stefan Monnier 2021-04-30 17:42 ` Thierry Volpiatto 0 siblings, 1 reply; 23+ messages in thread From: Stefan Monnier @ 2021-04-30 16:21 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: emacs-devel >> so I think before pushing to `externals/async` someone will have to >> merge the `elpa.git` changes into >> https://github.com/jwiegley/emacs-async. > > I will try to do it, if I can get async from elpa.git, don't > know how to get this for now. Assuming you're in a clone of `jwiegley/emacs-async`, you can do something like: git remote add -f -t externals/async elpa git://git.sv.gnu.org/emacs/elpa.git git merge elpa/externals/async -- Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 16:21 ` Stefan Monnier @ 2021-04-30 17:42 ` Thierry Volpiatto 2021-04-30 18:36 ` Stefan Monnier 2021-04-30 18:37 ` Thierry Volpiatto 0 siblings, 2 replies; 23+ messages in thread From: Thierry Volpiatto @ 2021-04-30 17:42 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel [-- Attachment #1: Type: text/plain, Size: 849 bytes --] Stefan Monnier <monnier@iro.umontreal.ca> writes: > [[PGP Encrypted Part:OK]] >>> so I think before pushing to `externals/async` someone will have to >>> merge the `elpa.git` changes into >>> https://github.com/jwiegley/emacs-async. >> >> I will try to do it, if I can get async from elpa.git, don't >> know how to get this for now. > > Assuming you're in a clone of `jwiegley/emacs-async`, you can do > something like: > > git remote add -f -t externals/async elpa git://git.sv.gnu.org/emacs/elpa.git > git merge elpa/externals/async Thanks, I could merge with many conflicts though, now I have to push, the address git://git.sv.gnu.org/emacs/elpa.git is the public address what is the equivalent with write access? <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? Just to be sure. Thanks. -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 17:42 ` Thierry Volpiatto @ 2021-04-30 18:36 ` Stefan Monnier 2021-04-30 18:42 ` Stefan Monnier 2021-04-30 18:37 ` Thierry Volpiatto 1 sibling, 1 reply; 23+ messages in thread From: Stefan Monnier @ 2021-04-30 18:36 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: emacs-devel > Thanks, I could merge with many conflicts though, now I have to push, > the address git://git.sv.gnu.org/emacs/elpa.git is the public address > what is the equivalent with write access? > > <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? That sounds right, yes, Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 18:36 ` Stefan Monnier @ 2021-04-30 18:42 ` Stefan Monnier 0 siblings, 0 replies; 23+ messages in thread From: Stefan Monnier @ 2021-04-30 18:42 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: emacs-devel >> Thanks, I could merge with many conflicts though, now I have to push, >> the address git://git.sv.gnu.org/emacs/elpa.git is the public address >> what is the equivalent with write access? >> >> <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? > > That sounds right, yes, So something like: git remote set-url elpa <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git followed by git push elpa master:externals/async should do the trick. Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 17:42 ` Thierry Volpiatto 2021-04-30 18:36 ` Stefan Monnier @ 2021-04-30 18:37 ` Thierry Volpiatto 2021-04-30 18:48 ` Stefan Monnier 1 sibling, 1 reply; 23+ messages in thread From: Thierry Volpiatto @ 2021-04-30 18:37 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel [-- Attachment #1: Type: text/plain, Size: 1156 bytes --] Thierry Volpiatto <thievol@posteo.net> writes: > [[PGP Signed Part:Good signature from 0EC56D141D16EF93 thievol@posteo.net (trust ultimate) created at 2021-04-30T19:47:29+0200 using RSA]] > > Stefan Monnier <monnier@iro.umontreal.ca> writes: > >> [[PGP Encrypted Part:OK]] >>>> so I think before pushing to `externals/async` someone will have to >>>> merge the `elpa.git` changes into >>>> https://github.com/jwiegley/emacs-async. >>> >>> I will try to do it, if I can get async from elpa.git, don't >>> know how to get this for now. >> >> Assuming you're in a clone of `jwiegley/emacs-async`, you can do >> something like: >> >> git remote add -f -t externals/async elpa git://git.sv.gnu.org/emacs/elpa.git >> git merge elpa/externals/async > > Thanks, I could merge with many conflicts though, now I have to push, > the address git://git.sv.gnu.org/emacs/elpa.git is the public address > what is the equivalent with write access? > > <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? So no, it is not this, it seems it create again an elpa branch, please remove it, sorry. I give up to complicated. -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 18:37 ` Thierry Volpiatto @ 2021-04-30 18:48 ` Stefan Monnier 2021-04-30 19:31 ` Thierry Volpiatto 0 siblings, 1 reply; 23+ messages in thread From: Stefan Monnier @ 2021-04-30 18:48 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: emacs-devel >> <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? > So no, it is not this, Yes it it, but this only specifies the repository, not the branch. In any case, if you push to John's `emacs-async` on Github, I can then bring it into elpa.git with a simple `make sync/async`. Actually, I just enabled the `:auto-sync` on this package, so that elpa.gnu.org will take care of pulling from that branch every once in a while (but that will not "merge" for you, so it won't work until the elpa.git branch is merged into https://github.com/jwiegley/emacs-async). Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 18:48 ` Stefan Monnier @ 2021-04-30 19:31 ` Thierry Volpiatto 2021-04-30 20:21 ` Stefan Monnier 0 siblings, 1 reply; 23+ messages in thread From: Thierry Volpiatto @ 2021-04-30 19:31 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel [-- Attachment #1: Type: text/plain, Size: 550 bytes --] Stefan Monnier <monnier@iro.umontreal.ca> writes: > [[PGP Encrypted Part:OK]] >>> <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? >> So no, it is not this, > > Yes it it, but this only specifies the repository, not the branch. Hmm, I understand nothing how the elpa is structured, too complicated for me. > In any case, if you push to John's `emacs-async` on Github, I can then > bring it into elpa.git with a simple `make sync/async`. Done, the branch to merge on elpa is the elpa branch, not master. Thanks. -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 19:31 ` Thierry Volpiatto @ 2021-04-30 20:21 ` Stefan Monnier 2021-05-01 4:45 ` Thierry Volpiatto 0 siblings, 1 reply; 23+ messages in thread From: Stefan Monnier @ 2021-04-30 20:21 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: emacs-devel >>>> <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? >>> So no, it is not this, >> Yes it it, but this only specifies the repository, not the branch. > Hmm, I understand nothing how the elpa is structured, too complicated for me. When you push a branch you need to tell to which branch in the remote repository you're pushing. The syntax is: git push <repository> <localbranch>:<remotebranch> so if you're pushing from your local `elpa` branch to elpa.git's `externals/async` branch, then you need to do: git push elpa elpa:externals/async I suspect the problem is that you used a shorthand which works well when the remote branch has the same name as a the local branch but can't be used here. I'm not sure how much simpler we could make it. E.g. if we used a separate repository per package, then presumably your `elpa` remote would point to something like git.sv.gnu.org:emacs/elpa/async.git and you'd need to push to the `main` branch in that repository, which would still bring the same problem since the name `main` is still different from the name `elpa` you used for your local branch. >> In any case, if you push to John's `emacs-async` on Github, I can then >> bring it into elpa.git with a simple `make sync/async`. > Done, the branch to merge on elpa is the elpa branch, not master. Any chance the two branches can be "unified"? AFAICT the difference are: - The `Package-Requires:` where `master` says ((emacs "24.3")), whereas `elpa` says ((cl-lib "0.5") (nadvice "0.3")). Not sure why they need to be different: we can "merge" them to ((emacs "24.3") (nadvice "0.3")), since cl-lib in included in Emacs-24.3 (OTOH nadvice appeared with Emacs-24.4, so I think requiring only Emacs-24.3 is not quite sufficient). - The `async-pkg.el` which is absent on `elpa` and present on `master`. Do we actually need it on `master`? Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-04-30 20:21 ` Stefan Monnier @ 2021-05-01 4:45 ` Thierry Volpiatto 2021-05-01 13:05 ` Yoni Rabkin 2021-05-01 13:06 ` updating async on elpa Stefan Monnier 0 siblings, 2 replies; 23+ messages in thread From: Thierry Volpiatto @ 2021-05-01 4:45 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel [-- Attachment #1: Type: text/plain, Size: 2405 bytes --] Stefan Monnier <monnier@iro.umontreal.ca> writes: > [[PGP Encrypted Part:OK]] >>>>> <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? >>>> So no, it is not this, >>> Yes it it, but this only specifies the repository, not the branch. >> Hmm, I understand nothing how the elpa is structured, too complicated for me. > > When you push a branch you need to tell to which branch in the remote > repository you're pushing. The syntax is: > > git push <repository> <localbranch>:<remotebranch> > > so if you're pushing from your local `elpa` branch to elpa.git's > `externals/async` branch, then you need to do: > > git push elpa elpa:externals/async > > I suspect the problem is that you used a shorthand which works well when > the remote branch has the same name as a the local branch but can't be > used here. I will rename it to something else. > I'm not sure how much simpler we could make it. E.g. if we used > a separate repository per package, then presumably your `elpa` remote > would point to something like git.sv.gnu.org:emacs/elpa/async.git > and you'd need to push to the `main` branch in that repository, which > would still bring the same problem since the name `main` is still > different from the name `elpa` you used for your local branch. Yes, the name I use -elpa- is bad, but yes a repository per repo would be great, in github organizations allow having one repo per package like in helm organization. >>> In any case, if you push to John's `emacs-async` on Github, I can then >>> bring it into elpa.git with a simple `make sync/async`. >> Done, the branch to merge on elpa is the elpa branch, not master. > > Any chance the two branches can be "unified"? > AFAICT the difference are: > > - The `Package-Requires:` where `master` says ((emacs "24.3")), whereas > `elpa` says ((cl-lib "0.5") (nadvice "0.3")). Not sure why they need > to be different: we can "merge" them to ((emacs "24.3") (nadvice "0.3")), > since cl-lib in included in Emacs-24.3 (OTOH nadvice appeared with > Emacs-24.4, so I think requiring only Emacs-24.3 is not quite sufficient). What about ((emacs "24.4")) we don't care of 24.3, we are at 27.2 now so very far from 24.3. > - The `async-pkg.el` which is absent on `elpa` and present on `master`. > Do we actually need it on `master`? IIRC yes, Melpa needs it. -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 4:45 ` Thierry Volpiatto @ 2021-05-01 13:05 ` Yoni Rabkin 2021-05-01 14:22 ` Stefan Monnier 2021-05-01 13:06 ` updating async on elpa Stefan Monnier 1 sibling, 1 reply; 23+ messages in thread From: Yoni Rabkin @ 2021-05-01 13:05 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: Stefan Monnier, emacs-devel Thierry Volpiatto <thievol@posteo.net> writes: > Stefan Monnier <monnier@iro.umontreal.ca> writes: > >> [[PGP Encrypted Part:OK]] >>>>>> <user>@git.sv.gnu.org:/srv/git/emacs/elpa.git ? >>>>> So no, it is not this, >>>> Yes it it, but this only specifies the repository, not the branch. >>> Hmm, I understand nothing how the elpa is structured, too complicated for me. >> >> When you push a branch you need to tell to which branch in the remote >> repository you're pushing. The syntax is: >> >> git push <repository> <localbranch>:<remotebranch> >> >> so if you're pushing from your local `elpa` branch to elpa.git's >> `externals/async` branch, then you need to do: >> >> git push elpa elpa:externals/async >> >> I suspect the problem is that you used a shorthand which works well when >> the remote branch has the same name as a the local branch but can't be >> used here. > > I will rename it to something else. > >> I'm not sure how much simpler we could make it. E.g. if we used >> a separate repository per package, then presumably your `elpa` remote >> would point to something like git.sv.gnu.org:emacs/elpa/async.git >> and you'd need to push to the `main` branch in that repository, which >> would still bring the same problem since the name `main` is still >> different from the name `elpa` you used for your local branch. > > Yes, the name I use -elpa- is bad, but yes a repository per repo would be > great, in github organizations allow having one repo per package like in > helm organization. Having one repository per package is how it works everywhere else (Savannah, etc.), and is easier to understand because of that. This is from the point of view of a contributor of course; I cannot speak to how it looks from Stefan's side. I would be much happier if emms and rt-liberation had their own repositories since it would mean that I couldn't possibly break anything except emms or rt-liberation when I push to those projects in elpa. As it stands, I'm always extremely wary that I would get the magical git incantation wrong and mess up the the u.s. constitution and the magna carta by mistake. -- "Cut your own wood and it will warm you twice" ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 13:05 ` Yoni Rabkin @ 2021-05-01 14:22 ` Stefan Monnier 2021-05-01 14:50 ` Thierry Volpiatto 2021-05-02 20:26 ` (was Re: updating async on elpa) auto-sync for emms Yoni Rabkin 0 siblings, 2 replies; 23+ messages in thread From: Stefan Monnier @ 2021-05-01 14:22 UTC (permalink / raw) To: Yoni Rabkin; +Cc: Thierry Volpiatto, emacs-devel > Having one repository per package is how it works everywhere else > (Savannah, etc.), and is easier to understand because of that. This is > from the point of view of a contributor of course; I cannot speak to how > it looks from Stefan's side. From my side it's plain and simply not an option at all because creating a new repository on git.sv.gnu.org requires manual intervention from the Savannah admins. Having had to design a solution that lives within this constraint, I'm actually pretty happy about the current result, so if the constraint were lifted, I'd probably want to try and preserve the advantages. In any case the `:auto-sync` feature should make it much less of an issue. > I would be much happier if emms and rt-liberation had their own > repositories since it would mean that I couldn't possibly break anything > except emms or rt-liberation when I push to those projects in elpa. The `elpa.git` is configured to reject non-fast-forward pushes, which should catch most blunders. Plus it is cloned at various different places which are not all constantly updated, plus (almost) all changes to it are reflected in the elpa-diffs mailing-list. So even if you do manage to make some really terrible mistake it should be quite easy to recover from it. You might want to activate the `:auto-sync` on your packages, tho to save you from the worry. Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 14:22 ` Stefan Monnier @ 2021-05-01 14:50 ` Thierry Volpiatto 2021-05-01 15:49 ` Stefan Monnier 2021-05-02 20:26 ` (was Re: updating async on elpa) auto-sync for emms Yoni Rabkin 1 sibling, 1 reply; 23+ messages in thread From: Thierry Volpiatto @ 2021-05-01 14:50 UTC (permalink / raw) To: Stefan Monnier; +Cc: Yoni Rabkin, emacs-devel [-- Attachment #1: Type: text/plain, Size: 1677 bytes --] Stefan Monnier <monnier@iro.umontreal.ca> writes: > [[PGP Encrypted Part:OK]] >> Having one repository per package is how it works everywhere else >> (Savannah, etc.), and is easier to understand because of that. This is >> from the point of view of a contributor of course; I cannot speak to how >> it looks from Stefan's side. > > From my side it's plain and simply not an option at all because creating > a new repository on git.sv.gnu.org requires manual intervention from the > Savannah admins. Having had to design a solution that lives within this > constraint, I'm actually pretty happy about the current result, so if > the constraint were lifted, I'd probably want to try and preserve > the advantages. > > In any case the `:auto-sync` feature should make it much less of an issue. > >> I would be much happier if emms and rt-liberation had their own >> repositories since it would mean that I couldn't possibly break anything >> except emms or rt-liberation when I push to those projects in elpa. > > The `elpa.git` is configured to reject non-fast-forward pushes, which > should catch most blunders. Plus it is cloned at various different > places which are not all constantly updated, plus (almost) all changes > to it are reflected in the elpa-diffs mailing-list. So even if you do > manage to make some really terrible mistake it should be quite easy to > recover from it. Oh that's good and make me more confident, like Yoni I am always afraid of corrupting the elpa repo. > You might want to activate the `:auto-sync` on your packages, tho to > save you from the worry. Is this documented somewhere? -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 14:50 ` Thierry Volpiatto @ 2021-05-01 15:49 ` Stefan Monnier 0 siblings, 0 replies; 23+ messages in thread From: Stefan Monnier @ 2021-05-01 15:49 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: Yoni Rabkin, emacs-devel >> You might want to activate the `:auto-sync` on your packages, tho to >> save you from the worry. > Is this documented somewhere? In the code and in my head, mostly. Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* (was Re: updating async on elpa) auto-sync for emms 2021-05-01 14:22 ` Stefan Monnier 2021-05-01 14:50 ` Thierry Volpiatto @ 2021-05-02 20:26 ` Yoni Rabkin 2021-05-03 13:39 ` Stefan Monnier 1 sibling, 1 reply; 23+ messages in thread From: Yoni Rabkin @ 2021-05-02 20:26 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel Stefan Monnier <monnier@iro.umontreal.ca> writes: > You might want to activate the `:auto-sync` on your packages, tho to > save you from the worry. Up until now I would merge into the "elpa" branch in the Emms Savannah repo, and then push to "externals/emms" from there. If I understand :auto-sync correctly (I recall us discussing such a feature), then I won't need the "elpa" branch in our repo anymore. Instead, we'll: * develop on "main" until we are ready for a release * increment ";; Version" * ...and emacs/elpa.git will do the rest (how often does it sync?). This assuming that nobody put any changes conflicting with the Emms Savannah repo directly into "externals/emms", which nobody should be doing anyway. So, can you please enable auto-sync for Emms as well so that it syncs from the main branch of our savannah repository? -- "Cut your own wood and it will warm you twice" ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: (was Re: updating async on elpa) auto-sync for emms 2021-05-02 20:26 ` (was Re: updating async on elpa) auto-sync for emms Yoni Rabkin @ 2021-05-03 13:39 ` Stefan Monnier 0 siblings, 0 replies; 23+ messages in thread From: Stefan Monnier @ 2021-05-03 13:39 UTC (permalink / raw) To: Yoni Rabkin; +Cc: emacs-devel >> You might want to activate the `:auto-sync` on your packages, tho to >> save you from the worry. > > Up until now I would merge into the "elpa" branch in the Emms Savannah > repo, and then push to "externals/emms" from there. If I understand > :auto-sync correctly (I recall us discussing such a feature), then I > won't need the "elpa" branch in our repo anymore. Instead, we'll: > > * develop on "main" until we are ready for a release > > * increment ";; Version" > > * ...and emacs/elpa.git will do the rest Sounds right. > (how often does it sync?). Currently the sync is done every hour, IIRC. > This assuming that nobody put any changes conflicting with > the Emms Savannah repo directly into "externals/emms", which > nobody should be doing anyway. Indeed, the auto-sync will only pull "fast-forwards" from the upstream, so if someone commits directly to `elpa.git` the auto-sync will fail until someone merges that changes into the upstream. > So, can you please enable auto-sync for Emms as well so that it syncs > from the main branch of our savannah repository? Done, Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 4:45 ` Thierry Volpiatto 2021-05-01 13:05 ` Yoni Rabkin @ 2021-05-01 13:06 ` Stefan Monnier 2021-05-01 14:38 ` Thierry Volpiatto 2021-05-01 15:37 ` Thierry Volpiatto 1 sibling, 2 replies; 23+ messages in thread From: Stefan Monnier @ 2021-05-01 13:06 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: emacs-devel > Yes, the name I use -elpa- is bad, Really? It sounds like a genuinely natural and intuitive choice to me ;-) > but yes a repository per repo would be great, in github organizations > allow having one repo per package like in helm organization. I know it's more traditional and has its advantages, but currently it's not really an option. > What about ((emacs "24.4")) we don't care of 24.3, we are at 27.2 now so > very far from 24.3. ((emacs "24.4")) would work, yes. Whether that's better than ((emacs "24.3") (nadvice "0.3")) or even ((cl-lib "0.5") (nadvice "0.3")) is for the maintainers of Async to decide ;-) >> - The `async-pkg.el` which is absent on `elpa` and present on `master`. >> Do we actually need it on `master`? > IIRC yes, Melpa needs it. Why would it matter, since it's on GNU ELPA anyway? Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 13:06 ` updating async on elpa Stefan Monnier @ 2021-05-01 14:38 ` Thierry Volpiatto 2021-05-01 15:37 ` Thierry Volpiatto 1 sibling, 0 replies; 23+ messages in thread From: Thierry Volpiatto @ 2021-05-01 14:38 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel [-- Attachment #1: Type: text/plain, Size: 1508 bytes --] Stefan Monnier <monnier@iro.umontreal.ca> writes: > [[PGP Encrypted Part:OK]] >> Yes, the name I use -elpa- is bad, > > Really? It sounds like a genuinely natural and intuitive choice to me ;-) So I misunderstood what you explained :-) I meant the branch name. >> but yes a repository per repo would be great, in github organizations >> allow having one repo per package like in helm organization. > > I know it's more traditional and has its advantages, but currently it's > not really an option. I understand it is not possible otherwise I am sure you would go this way as it is much simpler an safer for everybody. >> What about ((emacs "24.4")) we don't care of 24.3, we are at 27.2 now so >> very far from 24.3. > > ((emacs "24.4")) would work, yes. Whether that's better than > ((emacs "24.3") (nadvice "0.3")) or even ((cl-lib "0.5") (nadvice "0.3")) > is for the maintainers of Async to decide ;-) So lets go for ((emacs "24.4")). >>> - The `async-pkg.el` which is absent on `elpa` and present on `master`. >>> Do we actually need it on `master`? >> IIRC yes, Melpa needs it. > > Why would it matter, since it's on GNU ELPA anyway? As long as I have not a safe configuration for pushing to elpa I want to keep the melpa configuration i.e. with the pkg.el to be sure users use the last changes. For now each time I try to push to elpa something bad happens. Of course not having the need of a special branch for elpa would simplify. -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 13:06 ` updating async on elpa Stefan Monnier 2021-05-01 14:38 ` Thierry Volpiatto @ 2021-05-01 15:37 ` Thierry Volpiatto 2021-05-01 17:11 ` Stefan Monnier 1 sibling, 1 reply; 23+ messages in thread From: Thierry Volpiatto @ 2021-05-01 15:37 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel [-- Attachment #1: Type: text/plain, Size: 1360 bytes --] Hello Stefan again, now emacs-async master is compliant with elpa (package-requires, no pkg.el file), so now I can push from master to elpa, my remote called elpa is pointing to "me@git.sv$gnu.org:/srv/git/emacs/elpa.git", can I push safely with git push elpa master:refs/heads/externals/async? Thanks. Stefan Monnier <monnier@iro.umontreal.ca> writes: > [[PGP Encrypted Part:OK]] >> Yes, the name I use -elpa- is bad, > > Really? It sounds like a genuinely natural and intuitive choice to me ;-) > >> but yes a repository per repo would be great, in github organizations >> allow having one repo per package like in helm organization. > > I know it's more traditional and has its advantages, but currently it's > not really an option. > >> What about ((emacs "24.4")) we don't care of 24.3, we are at 27.2 now so >> very far from 24.3. > > ((emacs "24.4")) would work, yes. Whether that's better than > ((emacs "24.3") (nadvice "0.3")) or even ((cl-lib "0.5") (nadvice "0.3")) > is for the maintainers of Async to decide ;-) > >>> - The `async-pkg.el` which is absent on `elpa` and present on `master`. >>> Do we actually need it on `master`? >> IIRC yes, Melpa needs it. > > Why would it matter, since it's on GNU ELPA anyway? > > > Stefan > > > > > [[End of PGP Encrypted Part]] -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 15:37 ` Thierry Volpiatto @ 2021-05-01 17:11 ` Stefan Monnier 2021-05-01 18:01 ` Thierry Volpiatto 0 siblings, 1 reply; 23+ messages in thread From: Stefan Monnier @ 2021-05-01 17:11 UTC (permalink / raw) To: Thierry Volpiatto; +Cc: emacs-devel > now emacs-async master is compliant with elpa (package-requires, no > pkg.el file), so now I can push from master to elpa, my remote called > elpa is pointing to "me@git.sv$gnu.org:/srv/git/emacs/elpa.git", can I > push safely with git push elpa master:refs/heads/externals/async? Yes. Tho there's not much to push because the `:auto-sync` is enabled so elpa.gnu.org just pulled your new changes into elpa.git already. Stefan ^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: updating async on elpa 2021-05-01 17:11 ` Stefan Monnier @ 2021-05-01 18:01 ` Thierry Volpiatto 0 siblings, 0 replies; 23+ messages in thread From: Thierry Volpiatto @ 2021-05-01 18:01 UTC (permalink / raw) To: Stefan Monnier; +Cc: emacs-devel [-- Attachment #1: Type: text/plain, Size: 581 bytes --] Stefan Monnier <monnier@iro.umontreal.ca> writes: > [[PGP Encrypted Part:OK]] >> now emacs-async master is compliant with elpa (package-requires, no >> pkg.el file), so now I can push from master to elpa, my remote called >> elpa is pointing to "me@git.sv$gnu.org:/srv/git/emacs/elpa.git", can I >> push safely with git push elpa master:refs/heads/externals/async? > > Yes. Tho there's not much to push because the `:auto-sync` is enabled > so elpa.gnu.org just pulled your new changes into elpa.git already. Yes, I just realised this now, thanks. -- Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 686 bytes --] ^ permalink raw reply [flat|nested] 23+ messages in thread
end of thread, other threads:[~2021-05-03 13:39 UTC | newest] Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-04-30 11:56 updating async on elpa Thierry Volpiatto 2021-04-30 14:17 ` Stefan Monnier 2021-04-30 15:20 ` Thierry Volpiatto 2021-04-30 16:21 ` Stefan Monnier 2021-04-30 17:42 ` Thierry Volpiatto 2021-04-30 18:36 ` Stefan Monnier 2021-04-30 18:42 ` Stefan Monnier 2021-04-30 18:37 ` Thierry Volpiatto 2021-04-30 18:48 ` Stefan Monnier 2021-04-30 19:31 ` Thierry Volpiatto 2021-04-30 20:21 ` Stefan Monnier 2021-05-01 4:45 ` Thierry Volpiatto 2021-05-01 13:05 ` Yoni Rabkin 2021-05-01 14:22 ` Stefan Monnier 2021-05-01 14:50 ` Thierry Volpiatto 2021-05-01 15:49 ` Stefan Monnier 2021-05-02 20:26 ` (was Re: updating async on elpa) auto-sync for emms Yoni Rabkin 2021-05-03 13:39 ` Stefan Monnier 2021-05-01 13:06 ` updating async on elpa Stefan Monnier 2021-05-01 14:38 ` Thierry Volpiatto 2021-05-01 15:37 ` Thierry Volpiatto 2021-05-01 17:11 ` Stefan Monnier 2021-05-01 18:01 ` Thierry Volpiatto
Code repositories for project(s) associated with this public 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).