From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Benjamin Orthen Newsgroups: gmane.emacs.bugs Subject: bug#63916: 30.0.50; use-package: changes do not propagate to elpa-devel Date: Fri, 08 Sep 2023 12:55:38 +0200 Message-ID: References: <4b1ff1660351348f3d5f1bc848965d59@orthen.net> <874jnf3d0c.fsf@posteo.net> <87wn0a2sp8.fsf@posteo.net> <87fs6y6jkq.fsf@posteo.net> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-u9i+kT17SsRoEkMBV6cQ" Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="35654"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 63916@debbugs.gnu.org To: Stefan Monnier , Philip Kaludercic Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Fri Sep 08 12:56:26 2023 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qeZA1-000968-RB for geb-bug-gnu-emacs@m.gmane-mx.org; Fri, 08 Sep 2023 12:56:25 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qeZ9j-0005oE-7n; Fri, 08 Sep 2023 06:56:07 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qeZ9c-0005np-MR for bug-gnu-emacs@gnu.org; Fri, 08 Sep 2023 06:56:01 -0400 Original-Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1qeZ9c-0006iz-EQ for bug-gnu-emacs@gnu.org; Fri, 08 Sep 2023 06:56:00 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1qeZ9e-00047C-5e for bug-gnu-emacs@gnu.org; Fri, 08 Sep 2023 06:56:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Benjamin Orthen Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Fri, 08 Sep 2023 10:56:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 63916 X-GNU-PR-Package: emacs Original-Received: via spool by 63916-submit@debbugs.gnu.org id=B63916.169417054615797 (code B ref 63916); Fri, 08 Sep 2023 10:56:02 +0000 Original-Received: (at 63916) by debbugs.gnu.org; 8 Sep 2023 10:55:46 +0000 Original-Received: from localhost ([127.0.0.1]:42292 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qeZ9O-00046i-7I for submit@debbugs.gnu.org; Fri, 08 Sep 2023 06:55:46 -0400 Original-Received: from mailproxy01.manitu.net ([217.11.48.65]:50502) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qeZ9L-00046Z-FP for 63916@debbugs.gnu.org; Fri, 08 Sep 2023 06:55:44 -0400 In-Reply-To: X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:269748 Archived-At: --=-u9i+kT17SsRoEkMBV6cQ Content-Type: multipart/alternative; boundary="=-zmT79MLTb0ALixNJKJ1c" --=-zmT79MLTb0ALixNJKJ1c Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Related to this, I have a patch for elpa-admin.el (in the elpa repository) which would improve devel versioning for core packages: Instead of looking only at the git log of the main package file, we look instead at the git log of all files of the core package. This way, a new devel version is created when any other package file is changed, not just the main file. Is this the right mailing list to send the patch? If not, where could I send it to? Best, Benjamin On Sun, 2023-06-11 at 19:32 +0200, Benjamin Orthen wrote: > Hi, >=20 > I think I figured out why elpa does not build a new version of use- > package. >=20 > To determine the devel version, it looks only at the git log of > `(elpa- > -main-file pkg-spec)`, which in this case turns out to be use- > package.el. However, use-package.el has not been changed since > 20230115, I suppose this is why no new version is built. >=20 > Best, Benjamin >=20 >=20 > On Sun, 2023-06-11 at 12:55 -0400, Stefan Monnier wrote: > > > > I agree that these kinds of problems are annoying to debug.=C2=A0 I= t > > > > would be > > > > nice to offer a "debug mode" where the user can see what's > > > > going > > > > on > > > > within the sandbox.=C2=A0 Maybe we could do that by opening an > > > > interactive > > > > shell with a message saying "this is the command that we want > > > > to > > > > run" > > > > and then let the user run that command? > > > Shouldn't just invoking bash do that? > >=20 > > Yes, it should be fairly easy to do. > >=20 > > > But returning to the initial issue (hoping I did not miss > > > anything), if > > > the issue is not what I mentioned, then why is use-package not > > > building? > >=20 > > You did not miss anything: it's not building simply because the > > process > > run within the sandbox cannot read the target of the `use- > > package.texi` > > symlink because the sandbox does not expose this target. > >=20 > >=20 > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Stefan > >=20 >=20 --=-zmT79MLTb0ALixNJKJ1c Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: quoted-printable
Related to this,

I have = a patch for elpa-admin.el (in the elpa repository) which would improve deve= l versioning for core packages:

Instead of looking= only at the git log of the main package file, we look instead at the git l= og of all files of the core package.
This way, a new devel versio= n is created when any other package file is changed, not just the main file= .

Is this the right mailing list to send the patch= ? If not, where could I send it to?


Best,
Benjamin

On Sun, 2023-06-11 at 19= :32 +0200, Benjamin Orthen wrote:
Hi,<= br>

I think I figured out why elpa does not build = a new version of use-
package.

T= o determine the devel version, it looks only at the git log of `(elpa-
<= /div>
-main-file pkg-spec)`, which in this case turns out to be use-
package.el. However, use-package.el has not been changed since<= br>
20230115, I suppose this is why no new version is built.
<= /div>

Best, Benjamin


On Sun, 2023-06-11 at 12:55 -0400, Stefan Monnier wrote:
=
I agree that these kinds of problems are annoying to debug. = ; It
would be
nice to offer a "debug mode" wher= e the user can see what's going
on
within the s= andbox.  Maybe we could do that by opening an
interactiv= e
shell with a message saying "this is the command that we wa= nt to
run"
and then let the user run that comma= nd?
Shouldn't just invoking bash do that?

Yes, it should be fairly easy to do.

But returning to the = initial issue (hoping I did not miss
anything), if
<= div>the issue is not what I mentioned, then why is use-package not
building?

You did not miss = anything: it's not building simply because the
process
run within the sandbox cannot read the target of the `use-
package.texi`
symlink because the sandbox does not expo= se this target.


  &nb= sp;     Stefan



--=-zmT79MLTb0ALixNJKJ1c-- --=-u9i+kT17SsRoEkMBV6cQ Content-Disposition: attachment; filename*0=0001-Add-elpa-core-files-to-get-more-exact-devel-versions.pat; filename*1=ch Content-Type: text/x-patch; name="0001-Add-elpa-core-files-to-get-more-exact-devel-versions.patch"; charset="UTF-8" Content-Transfer-Encoding: base64 RnJvbSBkMjkxNjNlMDAzMTM2OTA0MzViMmJhYWNjNzcwYTczNDU5OGRkOTU2IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBCZW5qYW1pbiBPcnRoZW4gPGdpdEBvcnRoZW4ubmV0PgpEYXRl OiBGcmksIDggU2VwIDIwMjMgMTI6MDU6MTQgKzAyMDAKU3ViamVjdDogW1BBVENIXSBBZGQgZWxw YS0tY29yZS1maWxlcyB0byBnZXQgbW9yZSBleGFjdCBkZXZlbC12ZXJzaW9ucyBmb3IKIGNvcmUg cGFja2FnZXMKCi0tLQogZWxwYS1hZG1pbi5lbCB8IDUxICsrKysrKysrKysrKysrKysrKysrKysr KysrKysrKysrKysrKysrKystLS0tLS0tLS0tLQogMSBmaWxlIGNoYW5nZWQsIDQwIGluc2VydGlv bnMoKyksIDExIGRlbGV0aW9ucygtKQoKZGlmZiAtLWdpdCBhL2VscGEtYWRtaW4uZWwgYi9lbHBh LWFkbWluLmVsCmluZGV4IDJjMmQyYWVhYjcuLjU3ZGZlZThmMTggMTAwNjQ0Ci0tLSBhL2VscGEt YWRtaW4uZWwKKysrIGIvZWxwYS1hZG1pbi5lbApAQCAtOTA4LDIwICs5MDgsNDkgQEAgb2YgdGhl IGN1cnJlbnQgYHByb2Nlc3MtZW52aXJvbm1lbnQnLiAgUmV0dXJuIHRoZSBtb2RpZmllZCBjb3B5 LiIKICAgICAgICAgICAgICAgIDApKSkKICAgICAoZW5jb2RlLXRpbWUgKGxpc3QgcyBtaSBoIGQg bW8geSBuaWwgbmlsIHpzKSkpKQogCisoZGVmdW4gZWxwYWEtLWNvcmUtZmlsZXMgKHBrZy1zcGVj KQorICAiR2V0IGEgbGlzdCBvZiBjb3JlIGZpbGVzIChhbmQgb25seSBmaWxlcykgZm9yIFBLRy1T UEVDLgorQ29yZSBmb2xkZXJzIGFyZSByZWN1cnNpdmVseSBzZWFyY2hlZCwgZXhjbHVkZWQgZmls ZXMgYXJlIGlnbm9yZWQuIgorICAod2hlbi1sZXQKKyAgICAgICgoY29yZSAoZWxwYWEtLXNwZWMt Z2V0IHBrZy1zcGVjIDpjb3JlKSkpCisgICAgKGxldCoKKyAgICAgICAgKChleGNsdWRlcyAoZWxw YWEtLXNwZWMtZ2V0IHBrZy1zcGVjIDpleGNsdWRlcykpCisgICAgICAgICAoZW1hY3MtcmVwby1y b290IChleHBhbmQtZmlsZS1uYW1lICJlbWFjcyIpKQorICAgICAgICAgKGRlZmF1bHQtZGlyZWN0 b3J5IGVtYWNzLXJlcG8tcm9vdCkKKyAgICAgICAgIChmaWxlLXBhdHRlcm5zCisgICAgICAgICAg KGlmIChsaXN0cCBjb3JlKQorICAgICAgICAgICAgICBjb3JlCisgICAgICAgICAgICAobGlzdCBj b3JlKSkpCisgICAgICAgICAoY29yZS1maWxlcyBuaWwpKQorICAgICAgOzsgd2UgbG9vayBhdCBl YWNoIGZpbGUgb3IgZmlsZXMgaW4gZm9sZGVyIGFuZCBhZGQgdGhlbQorICAgICAgOzsgdG8gY29y ZS1maWxlcyBpZiB0aGV5IGFyZSBpbiB0aGUgZXhjbHVkZXMKKyAgICAgIChjbC1sYWJlbHMgKChw cm9jZXNzLWl0ZW0gKGl0ZW0pCisgICAgICAgICAgICAgICAgICAgICh1bmxlc3MgKG1lbWJlciBp dGVtIGV4Y2x1ZGVzKQorICAgICAgICAgICAgICAgICAgICAgIChpZiAoZmlsZS1kaXJlY3Rvcnkt cCBpdGVtKQorICAgICAgICAgICAgICAgICAgICAgICAgICAoZG9saXN0IChmaWxlIChkaXJlY3Rv cnktZmlsZXMgaXRlbSBuaWwgZGlyZWN0b3J5LWZpbGVzLW5vLWRvdC1maWxlcy1yZWdleHApKQor ICAgICAgICAgICAgICAgICAgICAgICAgICAgIChwcm9jZXNzLWl0ZW0gKGNvbmNhdCBpdGVtIGZp bGUpKSkKKyAgICAgICAgICAgICAgICAgICAgICAgIChwdXNoIGl0ZW0gY29yZS1maWxlcykpKSkp CisgICAgICAgIChkb2xpc3QgKGl0ZW0gZmlsZS1wYXR0ZXJucykKKyAgICAgICAgICAocHJvY2Vz cy1pdGVtIGl0ZW0pKSkKKyAgICAgIGNvcmUtZmlsZXMpKSkKKwogKGRlZnVuIGVscGFhLS1nZXQt ZGV2ZWwtdmVyc2lvbiAoZGlyIHBrZy1zcGVjKQogICAiQ29tcHV0ZSB0aGUgZGF0ZS1iYXNlZCBw c2V1ZG8tdmVyc2lvbiB1c2VkIGZvciBkZXZlbCBidWlsZHMuIgotICAobGV0KiAoKGZ0biAoZmls ZS10cnVlbmFtZSAgICAgIDs7IEZvbGxvdyBzeW1saW5rcyEKLSAgICAgICAgICAgICAgKGV4cGFu ZC1maWxlLW5hbWUgKGVscGFhLS1tYWluLWZpbGUgcGtnLXNwZWMpIGRpcikpKQotICAgICAgICAg KGRlZmF1bHQtZGlyZWN0b3J5IChmaWxlLW5hbWUtZGlyZWN0b3J5IGZ0bikpCi0gICAgICAgICAo Z2l0ZGF0ZQorICAobGV0KiAoKGdpdGRhdGUKICAgICAgICAgICAod2l0aC10ZW1wLWJ1ZmZlcgot ICAgICAgICAgICAoaWYgKHBsaXN0LWdldCAoY2RyIHBrZy1zcGVjKSA6Y29yZSkKLSAgICAgICAg ICAgICAgIDs7IEZvciBjb3JlIHBhY2thZ2VzLCBkb24ndCB1c2UgdGhlIGRhdGUgb2YgdGhlIGxh c3QKLSAgICAgICAgICAgICAgIDs7IGNvbW1pdCB0byB0aGUgYnJhbmNoLCBidXQgdGhhdCBvZiB0 aGUgbGFzdCBjb21taXQKLSAgICAgICAgICAgICAgIDs7IHRvIHRoZSBtYWluIGZpbGUuCi0gICAg ICAgICAgICAgICAoZWxwYWEtLWNhbGwgdCAiZ2l0IiAibG9nIiAiLS1wcmV0dHk9Zm9ybWF0OiVj SSIgIi0tbm8tcGF0Y2giCi0gICAgICAgICAgICAgICAgICAgICAgICAgICAgIi0xIiAiLS0iIChm aWxlLW5hbWUtbm9uZGlyZWN0b3J5IGZ0bikpCi0gICAgICAgICAgICAgKGVscGFhLS1jYWxsIHQg ImdpdCIgInNob3ciICItLXByZXR0eT1mb3JtYXQ6JWNJIiAiLS1uby1wYXRjaCIpKQorICAgICAg ICAgICAgKGlmIChwbGlzdC1nZXQgKGNkciBwa2ctc3BlYykgOmNvcmUpCisgICAgICAgICAgICAg ICAgKGxldAorICAgICAgICAgICAgICAgICAgICAoKGNvcmUtZmlsZXMgKGVscGFhLS1jb3JlLWZp bGVzIHBrZy1zcGVjKSkKKyAgICAgICAgICAgICAgICAgICAgIChkZWZhdWx0LWRpcmVjdG9yeSAo ZXhwYW5kLWZpbGUtbmFtZSAiZW1hY3MiKSkpCisgICAgICAgICAgICAgICAgICA7OyBGb3IgY29y ZSBwYWNrYWdlcywgZG9uJ3QgdXNlIHRoZSBkYXRlIG9mIHRoZSBsYXN0CisgICAgICAgICAgICAg ICAgICA7OyBjb21taXQgdG8gdGhlIGJyYW5jaCwgYnV0IHRoYXQgb2YgdGhlIGxhc3QgY29tbWl0 CisgICAgICAgICAgICAgICAgICA7OyB0byB0aGUgY29yZSBmaWxlcy4KKyAgICAgICAgICAgICAg ICAgIChhcHBseSAnZWxwYWEtLWNhbGwgdCAiZ2l0IiAibG9nIiAiLS1wcmV0dHk9Zm9ybWF0OiVj SSIgIi0tbm8tcGF0Y2giCisgICAgICAgICAgICAgICAgICAgICAgICAgIi0xIiAiLS0iIGNvcmUt ZmlsZXMpKQorICAgICAgICAgICAgICAobGV0KiAoKGZ0biAoZmlsZS10cnVlbmFtZSAgICAgIDs7 IEZvbGxvdyBzeW1saW5rcyEKKyAgICAgICAgICAgICAgICAgICAgICAgICAgIChleHBhbmQtZmls ZS1uYW1lIChlbHBhYS0tbWFpbi1maWxlIHBrZy1zcGVjKSBkaXIpKSkKKyAgICAgICAgICAgICAg ICAgICAgIChkZWZhdWx0LWRpcmVjdG9yeSAoZmlsZS1uYW1lLWRpcmVjdG9yeSBmdG4pKSkKKyAg ICAgICAgICAgICAgICAoZWxwYWEtLWNhbGwgdCAiZ2l0IiAic2hvdyIgIi0tcHJldHR5PWZvcm1h dDolY0kiICItLW5vLXBhdGNoIikpKQogICAgICAgICAgICAgKGJ1ZmZlci1zdHJpbmcpKSkKICAg ICAgICAgICh2ZXJkYXRlCiAgICAgICAgICAgOzsgQ29udmVydCBHaXQncyBkYXRlIGludG8gc29t ZXRoaW5nIHRoYXQgbG9va3MgbGlrZSBhIHZlcnNpb24gbnVtYmVyLgotLSAKMi40MS4wCgo= --=-u9i+kT17SsRoEkMBV6cQ--