bug#75328: 31.0.50; use-package cannot override a built in package

bug#75328: 31.0.50; use-package cannot override a built in package

[Mark A. Hershberger via Bug reports for GNU Emacs, the Swiss army knife of text editors]

The following command sequence does not check out transient into
~/.emacs.d/elpa/transient while a similar sequence for magit checks out
magit:

    $ rm -rf ~/.emacs.d/
    $ echo '(use-package transient
           :vc (:url "https://github.com/magit/transient") :ensure t)' > .emacs
    $ emacs

Since the bundled transient cannot be used with forge from git, I ended
up installing transient using package-install.

I looked over help for use-package, but I cannot see a way to force it
to use the version specified in :vc for the package instead of the
bundled version.

Thanks,

Mark.

bug#75328: 31.0.50; use-package cannot override a built in package

[Eli Zaretskii]

> Date: Fri, 03 Jan 2025 17:29:51 -0500
> From:  "Mark A. Hershberger" via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
> 
> 
> The following command sequence does not check out transient into
> ~/.emacs.d/elpa/transient while a similar sequence for magit checks out
> magit:
> 
>     $ rm -rf ~/.emacs.d/
>     $ echo '(use-package transient
>            :vc (:url "https://github.com/magit/transient") :ensure t)' > .emacs
>     $ emacs
> 
> Since the bundled transient cannot be used with forge from git, I ended
> up installing transient using package-install.
> 
> I looked over help for use-package, but I cannot see a way to force it
> to use the version specified in :vc for the package instead of the
> bundled version.

Did you try customizing package-install-upgrade-built-in?

bug#75328: 31.0.50; use-package cannot override a built in package

[Mark A. Hershberger via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Eli Zaretskii <eliz@gnu.org> writes:

> Did you try customizing package-install-upgrade-built-in?

Yes.  I also reported this to Jonas[1] and in the following discussion
he suggested that and I tried it[2].

The following does not check out the transient repository into
~/.emacs.d/elpa/:

    rm -rf ~/.emacs.d &&
    echo '(setq package-install-upgrade-built-in t)
    (use-package transient :vc (:url "https://github.com/magit/transient" :rev :newest) :ensure t)' > ~/.emacs &&
    ~/bin/emacs

Footnotes:
[1]  https://github.com/magit/forge/issues/732

[2]  https://github.com/magit/forge/discussions/733#discussioncomment-11733964

bug#75328: 31.0.50; use-package cannot override a built in package

[Eli Zaretskii]

> From: "Mark A. Hershberger" <mah@everybody.org>
> Cc: 75328@debbugs.gnu.org
> Date: Sat, 04 Jan 2025 10:50:21 -0500
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > Did you try customizing package-install-upgrade-built-in?
> 
> Yes.  I also reported this to Jonas[1] and in the following discussion
> he suggested that and I tried it[2].
> 
> The following does not check out the transient repository into
> ~/.emacs.d/elpa/:
> 
>     rm -rf ~/.emacs.d &&
>     echo '(setq package-install-upgrade-built-in t)
>     (use-package transient :vc (:url "https://github.com/magit/transient" :rev :newest) :ensure t)' > ~/.emacs &&
>     ~/bin/emacs

Philip, shouldn't package-install-upgrade-built-in allow upgrading a
built-in package via use--package's :vc keyword?

bug#75328: 31.0.50; use-package cannot override a built in package

[Philip Kaludercic]

Eli Zaretskii <eliz@gnu.org> writes:

>> From: "Mark A. Hershberger" <mah@everybody.org>
>> Cc: 75328@debbugs.gnu.org
>> Date: Sat, 04 Jan 2025 10:50:21 -0500
>> 
>> Eli Zaretskii <eliz@gnu.org> writes:
>> 
>> > Did you try customizing package-install-upgrade-built-in?
>> 
>> Yes.  I also reported this to Jonas[1] and in the following discussion
>> he suggested that and I tried it[2].
>> 
>> The following does not check out the transient repository into
>> ~/.emacs.d/elpa/:
>> 
>>     rm -rf ~/.emacs.d &&
>>     echo '(setq package-install-upgrade-built-in t)
>>     (use-package transient :vc (:url "https://github.com/magit/transient" :rev :newest) :ensure t)' > ~/.emacs &&
>>     ~/bin/emacs
>
> Philip, shouldn't package-install-upgrade-built-in allow upgrading a
> built-in package via use--package's :vc keyword?

Built-in packages are not supported by package-vc right now, as
emacs.git is not a emacs package the way ELPA intends it.  There is also
the annoyance that without custom code (which would be doable if we had
to) we would have a separate checkout of emacs.git for every built-in
package that one wishes to clone that way.  This would be a lot easier
if Git would support subtree or single-file checkouts...

bug#75328: 31.0.50; use-package cannot override a built in package

[Eli Zaretskii]

> From: Philip Kaludercic <philipk@posteo.net>
> Cc: "Mark A. Hershberger" <mah@everybody.org>,  75328@debbugs.gnu.org
> Date: Sat, 04 Jan 2025 21:57:18 +0000
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> The following does not check out the transient repository into
> >> ~/.emacs.d/elpa/:
> >> 
> >>     rm -rf ~/.emacs.d &&
> >>     echo '(setq package-install-upgrade-built-in t)
> >>     (use-package transient :vc (:url "https://github.com/magit/transient" :rev :newest) :ensure t)' > ~/.emacs &&
> >>     ~/bin/emacs
> >
> > Philip, shouldn't package-install-upgrade-built-in allow upgrading a
> > built-in package via use--package's :vc keyword?
> 
> Built-in packages are not supported by package-vc right now, as
> emacs.git is not a emacs package the way ELPA intends it.  There is also
> the annoyance that without custom code (which would be doable if we had
> to) we would have a separate checkout of emacs.git for every built-in
> package that one wishes to clone that way.  This would be a lot easier
> if Git would support subtree or single-file checkouts...

So we should convert this bug into a feature-request, and meanwhile
document this restriction in the use-package manual?

bug#75328: 31.0.50; use-package cannot override a built in package

[Philip Kaludercic]

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Philip Kaludercic <philipk@posteo.net>
>> Cc: "Mark A. Hershberger" <mah@everybody.org>,  75328@debbugs.gnu.org
>> Date: Sat, 04 Jan 2025 21:57:18 +0000
>> 
>> Eli Zaretskii <eliz@gnu.org> writes:
>> 
>> >> The following does not check out the transient repository into
>> >> ~/.emacs.d/elpa/:
>> >> 
>> >>     rm -rf ~/.emacs.d &&
>> >>     echo '(setq package-install-upgrade-built-in t)
>> >>     (use-package transient :vc (:url "https://github.com/magit/transient" :rev :newest) :ensure t)' > ~/.emacs &&
>> >>     ~/bin/emacs
>> >
>> > Philip, shouldn't package-install-upgrade-built-in allow upgrading a
>> > built-in package via use--package's :vc keyword?
>> 
>> Built-in packages are not supported by package-vc right now, as
>> emacs.git is not a emacs package the way ELPA intends it.  There is also
>> the annoyance that without custom code (which would be doable if we had
>> to) we would have a separate checkout of emacs.git for every built-in
>> package that one wishes to clone that way.  This would be a lot easier
>> if Git would support subtree or single-file checkouts...
>
> So we should convert this bug into a feature-request, and meanwhile
> document this restriction in the use-package manual?

If anything, we should document this in (emacs) Fetching Package
Sources, but one can mention that in the use-package manual as well.