Move metadata from ego-pkg.el to ego.el #112
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull-request moves metadata about this package from
<name>-pkg.el
to<name>.el
and deletes the now redundant<name>-pkg.el
.The
<name>-pkg.el
file is not supposed to be tracked in the Git repository. It is supposed to be generated by the package archive (e.g., MELPA) and distributed as part of the<name>-<version>.tar
file.If you are already convinced, you can skip reading most of the rest, but please read the last page.
Unfortunately the Multi-file Packages node in the info manual is not explicit about this. So you may have read this:
... and concluded that this means that your Git repository must contain such a
<name>-pkg.el
file. If so, please carefully read at least the preceding paragraph again (emphasize mine):This node talks about how a package archive (such as GNU ELPA or MELPA) should distribute "multi-file" packages as tarballs, and goes into the details of what that tarball must contain. It also mentions how that tarball is unpacked when a user installs the package.
This node does not describe what files must exist in repositories used to develop individual packages. It does not say, that such repositories must contain a
<name>-pkg.el
file. (But granted, it also does not say, that they must not contain such a file.)Content directory refers to a directory that was created by unpacking a tarfile. It does not refer to a Git repository.
The documentation lacks guidance on whether the repositories, used to maintain individual packages, should, or should not, track
<name>-pkg.el
. So we have to turn elsewhere to answer that question.How do, or don't, the tools used to maintain the well known package archives use
<name>-pkg.el
as a source of information? (They obviously all generate such a file, because that is a requirement, as we have seen above.)elpa-admin.el
, the tool used to maintain the "official" package archives GNU ELPA and NonGNU ELPA, completely ignore any checked in<name>-pkg.el
. It has always done that.package-build.el
, the tool used to maintain the much older MELPA, gets metadata from<name>.el
, but iff some information is missing from that file and<name>-pkg.el
exists and contains that piece of information, then that is used.Melpa has existed for a long time and when it was young, many conventions had not been established yet, and so
package-build.el
had to deal with a complex reality of many different, and often conflicting or otherwise problematic practices.So the reason that
package-build.el
falls back to using information from<name>-pkg.el
is not that its maintainer (that would be me) thinks that doing so is a good idea, but that it was once necessary and unfortunately still is.But now that only 1% of packages use
<name>-pkg.el
without providing the same information in<name>.el
also (or preferable only), I am opening this pull-request, and four dozen like it, so thatpackage-build.el
can soon finally stop getting information from that file.So why is it problematic to use
<name>-pkg.el
in the first place?When the same information can be specified in
<name>.el
and<name>-pkg.el
, then there are two sources of truth. This is not a problem as long as everybody always makes sure the information is updated in both places. Unfortunately, but not at all surprisingly, that very often is not the case.Many, if not most, package maintainers who initially put the information in both places, over time forget about one of them and stop updating it. To complicate that further, while most of them seem to stop updating
<name>-pkg.el
, some instead forget to update<name>.el
. For a tool likepackage-build.el
, that makes it impossible to always pick the correct source of truth, as intended by the package maintainer.I believe the only solution to this problem is to agree on a single source of truth. As a community we have all but done so already. Among the 5837 packages on MELPA, 99% provide the expected information in
<name>.el
, while only %2 provide it in<name>-pkg.el
.This pull-request, and four dozen others like it, deal with the %1 that only provide this information in
<name>-pkg.el
. Once these have been merged, MELPA/package-build.el
, can start ignoring<name>-pkg.el
, just like GNU ELPA and NonGNU ELPA have always done.Once that has happened, the focus can turn to informing the package maintainers, who provide inconsistent information in the two places, that they are doing so, and might want to remove
<name>-pkg.el
, to prevent that going forward.Merging this pull-request addresses the problem for "regular" MELPA, the channel that distributes package snapshots.
But doing this alone, does not address the problem for MELPA Stable. In the commit from which the release version is build,
<name>-pkg.el
still exists and the information is still missing from<name>.el
.So please also create a new release!
If you also tag your releases using Git, then please make sure you bump the version in both places. I.e., bump the
Package-Version
library header, create a new commit titled something like "Release version N", and do not forget to also tag that exact commit with the matchingN
orvN
git tag.Thanks for you attention! Jonas
Ps: Please double check this pull-request before merging. As I have mentioned, I am opening four dozen such pull-requests, and it is quite possible that I made a copy-paste error in one or two.