unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Philip Kaludercic <philipk@posteo.net>
To: Suhail Singh <suhailsingh247@gmail.com>
Cc: "Martin Edström" <meedstrom@runbox.eu>,
	emacs-devel <emacs-devel@gnu.org>,
	"Tony Zorman" <tonyzorman@mailbox.org>
Subject: Re: Reconsider defaults for use-package-vc-prefer-newest
Date: Fri, 20 Sep 2024 20:34:11 +0000	[thread overview]
Message-ID: <87setum5do.fsf@posteo.net> (raw)
In-Reply-To: <87msk3jr0u.fsf@gmail.com> (Suhail Singh's message of "Thu, 19 Sep 2024 17:02:41 -0400")

Suhail Singh <suhailsingh247@gmail.com> writes:

> Philip Kaludercic <philipk@posteo.net> writes:
>
>>> Would it be possible to reflect the commit corresponding to the version
>>> in the "Other versions:" field in the output of M-x describe-package ?
>>
>> Why under "Other version:" (which links to other packages).
>
> For a package that is available under both MELPA Stable and NonGNU ELPA
> one can encounter the situation where the versions look the same, but
> instead point to different commits.  They don't necessarily have to
> point to different commits, and in fact when they do I would consider it
> a bug.  However, as of today, a user only becomes aware of this if they
> follow the links in the "Other version" and then contrast the "Commit"
> information in the subsequent package's description with the one from
> the prior page.
>
> An example of a package where this was observed in the wild (probably
> not the only case) was julia-mode.  The 0.4 version of that package on
> NonGNU corresponds to commit 6408b96c1c97e41bc2af060d661afee4f7b22e89
> whereas the version 0.4 on MELPA Stable corresponds to commit
> 8bfc709716a257521cb386f20b8932e83db930a9 .
>
> Since the information is available via M-x button-describe on the links
> to the other packages, hopefully it could be made visible (perhaps via
> some customization).

One concern I have would be that an entire revision string can get very
cluttered, and is usually not interesting (as I don't use third-party
archives, I don't have duplicates of the kind that you describe).  I
wonder if indicating a "commit mismatch" for remote packages might be
interesting (we explicitly don't want this for local packages,
e.g. packages installed via package-vc).

Could you perhaps elaborate on why you consider this to be a bug?

>> IIRC the commit of a VC package should appear under the commit header.
>
> That necessitates additional clicks.  If those clicks or button pushes
> could be avoided it would improve user convenience.

True, but there are many things that could be made more convenient that
only interest few people.  I am trying to understand why this is
something that would interest everyone.

>> There is the possibility of using a symbolic name if available.
>
> Could you please give an example?

What I had in mind was using `vc-git-symbolic-commit'.  So in the case
of Git: If the commit
is following the "master" branch, then it would print "master" instead
a commit hash, likewise if the package is checked out on some tag, it
could display that.

>>> Additionally, if it were possible to reflect the commit date as well,
>>> that would be helpful as well.
>>
>> I don't think there is a nice vc-generic way to extract the date of a
>> commit, at least without extending the vc interface any further.  I
>> might be mistaken though.
>
> I may be mistaken, but my understanding was that if `annotate-command'
> is defined for a backend then this may be possible via `annotate-time'.

From what I see, that would only give us "the time of the next line of
annotation at or after point, as a floating point fractional number
of days.", and not something we can cleanly use to get an exact date.  I
mean you could try to multiply it with (* 60 60 24) to get the value
that was probably passed to `vc-annotate-convert-time', but that feels
unreliable to me.  But I too might be mistaken.

>> The other issues is that if you only have the revision string from the
>> ELPA server, without any checkout, it isn't easy/cheap to determine the
>> date even if you just wanted to support Git.
>
> Ah.  That is, indeed, a problem.  Out of curiosity, what is an example
> of such a revision string (say, for the julia-mode package from NonGNU
> ELPA)?

In the case of Git it is just a commit hash, like the ones you gave above?

-- 
	Philip Kaludercic on siskin



  reply	other threads:[~2024-09-20 20:34 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-14 12:09 Reconsider defaults for use-package-vc-prefer-newest Martin Edström
2024-09-19 11:49 ` Philip Kaludercic
2024-09-19 18:50   ` Suhail Singh
2024-09-19 20:11     ` Philip Kaludercic
2024-09-19 20:31       ` Philip Kaludercic
2024-09-20  6:15         ` Eli Zaretskii
2024-09-20 15:14           ` Philip Kaludercic
2024-09-20 15:45             ` Eli Zaretskii
2024-09-20 16:56               ` Philip Kaludercic
2024-09-20 17:37                 ` Eli Zaretskii
2024-09-20 20:43               ` Philip Kaludercic
2024-09-21  7:14                 ` Eli Zaretskii
2024-09-21 14:31                   ` Philip Kaludercic
2024-09-21 15:18                     ` Eli Zaretskii
2024-09-21 15:43                       ` Philip Kaludercic
2024-09-19 21:02       ` Suhail Singh
2024-09-20 20:34         ` Philip Kaludercic [this message]
2024-09-20 23:38           ` Suhail Singh
2024-09-21  7:06             ` chad
2024-09-21 14:27               ` Suhail Singh
2024-09-21 15:59             ` Philip Kaludercic
2024-09-21 19:04               ` Suhail Singh
2024-09-22 15:30                 ` Philip Kaludercic
2024-09-22 20:08                   ` Suhail Singh
2024-09-25 12:06                     ` Suhail Singh
2024-09-25 13:15                       ` Philip Kaludercic
2024-09-25 12:07                     ` Philip Kaludercic
2024-09-25 15:15                       ` Suhail Singh
2024-09-25 20:11                         ` Philip Kaludercic
2024-09-25 20:48                           ` Suhail Singh
2024-09-29  2:13                           ` Richard Stallman
2024-09-29  7:25                             ` Philip Kaludercic
2024-09-29 13:55                               ` Suhail Singh
2024-09-26 23:23                     ` Charles Choi
2024-09-27  0:17                       ` Adam Porter
2024-09-27  0:33                       ` Suhail Singh
2024-09-27  6:46                       ` Eli Zaretskii
2024-09-29 14:22                         ` Stefan Kangas
2024-09-29 14:35                           ` Eli Zaretskii
2024-10-11 18:54                         ` Suhail Singh
2024-10-12  5:46                           ` Eli Zaretskii
2024-09-20  4:57   ` Tony Zorman
2024-09-20 19:37     ` Martin Edström
2024-09-20 21:05       ` Philip Kaludercic
2024-09-21 14:44     ` Philip Kaludercic
2024-09-21 14:58       ` Tony Zorman
2024-09-21 15:10       ` Suhail Singh
2024-09-21 16:09         ` Philip Kaludercic
  -- strict thread matches above, loose matches on Subject: below --
2024-09-15 17:38 Martin Edström
2024-09-15 18:24 ` Eli Zaretskii
2024-09-15 19:46   ` Martin Edstrom
2024-09-16 11:34     ` Eli Zaretskii
2024-09-16 15:24       ` Martin Edström
2024-09-16 16:15       ` Martin Edström
2024-09-16 17:57         ` Eli Zaretskii
2024-09-18 14:30           ` Martin Edström
2024-09-15 19:52 Martin Edström
2024-09-15 20:41 ` chad
2024-09-15 21:09   ` Martin Edstrom
2024-09-15 22:12     ` chad
2024-09-15 23:51       ` Martin Edstrom
2024-09-16 11:50     ` Eli Zaretskii
2024-09-16 16:46       ` Martin Edström
2024-09-16 18:10         ` Eli Zaretskii
2024-09-16 20:16       ` Suhail Singh
2024-09-17 11:44         ` Eli Zaretskii
2024-09-19  3:38           ` Suhail Singh
2024-09-19  6:28             ` Eli Zaretskii
2024-09-19 12:08               ` Suhail Singh
2024-09-19 12:39                 ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87setum5do.fsf@posteo.net \
    --to=philipk@posteo.net \
    --cc=emacs-devel@gnu.org \
    --cc=meedstrom@runbox.eu \
    --cc=suhailsingh247@gmail.com \
    --cc=tonyzorman@mailbox.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).