(→Usage) |
|||
Line 58: | Line 58: | ||
'''Version:''' 2.2.5 | '''Version:''' 2.2.5 | ||
'''%forgemeta' | '''%forgemeta -i''' | ||
… | … | ||
Line 70: | Line 69: | ||
'''%forgesetup''' | '''%forgesetup''' | ||
'' | ''$ rpmbuild -bs" output: | ||
Forge-specific packaging variables | Forge-specific packaging variables | ||
forgeurl: https://github.com/alecthomas/kingpin | forgeurl: https://github.com/alecthomas/kingpin | ||
forgesource: | forgesource: https://github.com/alecthomas/kingpin/archive/v2.2.5.tar.gz#/kingpin-2.2.5.tar.gz | ||
forgesetupargs: -n | forgesetupargs: -n kingpin-2.2.5 | ||
Generic variables | Generic variables | ||
archivename: kingpin- | archivename: kingpin-2.2.5 | ||
archiveext: tar.gz | archiveext: tar.gz | ||
archiveurl: | archiveurl: https://github.com/alecthomas/kingpin/archive/v2.2.5.tar.gz | ||
scm: | scm: | ||
tag: | tag: | ||
commit: | commit: | ||
dist: . | dist: .fc28 (snapshot date is computed once %{_sourcedir}/%{archivename}.%{archiveext} is available) | ||
=== Packaging a release (GitLab) === | === Packaging a release (GitLab) === | ||
Line 114: | Line 112: | ||
'''%global tag''' %{version} | '''%global tag''' %{version} | ||
'''%forgemeta' | '''%forgemeta -i''' | ||
… | … | ||
Line 126: | Line 123: | ||
'''%forgesetup''' | '''%forgesetup''' | ||
'' | ''$ rpmbuild -bs" output: | ||
Forge-specific packaging variables | Forge-specific packaging variables | ||
forgeurl: https://github.com/apache/thrift | forgeurl: https://github.com/apache/thrift | ||
forgesource: https://github.com/apache/thrift/archive/ | forgesource: https://github.com/apache/thrift/archive/0.10.0.tar.gz#/thrift-0.10.0.tar.gz | ||
forgesetupargs: -n | forgesetupargs: -n thrift-0.10.0 | ||
Generic variables | Generic variables | ||
archivename: thrift- | archivename: thrift-0.10.0 | ||
archiveext: tar.gz | archiveext: tar.gz | ||
archiveurl: https://github.com/apache/thrift/archive/ | archiveurl: https://github.com/apache/thrift/archive/0.10.0.tar.gz | ||
scm: git | scm: git | ||
tag: 0.10.0 | tag: 0.10.0 | ||
commit: | commit: | ||
dist: . | dist: .fc28 (snapshot date is computed once %{_sourcedir}/%{archivename}.%{archiveext} is available) | ||
=== Packaging a commit === | === Packaging a commit === | ||
Line 146: | Line 142: | ||
'''%global commit''' 3a1d936b7575b82197a1fea0632218dd07b1e65c | '''%global commit''' 3a1d936b7575b82197a1fea0632218dd07b1e65c | ||
'''%forgemeta' | '''%forgemeta -i''' | ||
… | … | ||
Line 159: | Line 154: | ||
'''%forgesetup''' | '''%forgesetup''' | ||
''' | ''' | ||
'' | |||
''$ rpmbuild -bs" output: | |||
Forge-specific packaging variables | Forge-specific packaging variables | ||
forgeurl: https://code.googlesource.com/google-api-go-client | forgeurl: https://code.googlesource.com/google-api-go-client | ||
forgesource: | forgesource: https://code.googlesource.com/google-api-go-client/+archive/3a1d936b7575b82197a1fea0632218dd07b1e65c.tar.gz#/google-api-go-client-3a1d936b7575b82197a1fea0632218dd07b1e65c.tar.gz | ||
forgesetupargs: -c | forgesetupargs: -c | ||
Generic variables | Generic variables | ||
archivename: google-api-go-client- | archivename: google-api-go-client-3a1d936b7575b82197a1fea0632218dd07b1e65c | ||
archiveext: tar.gz | archiveext: tar.gz | ||
archiveurl: | archiveurl: https://code.googlesource.com/google-api-go-client/+archive/3a1d936b7575b82197a1fea0632218dd07b1e65c.tar.gz | ||
scm: git | scm: git | ||
tag: | tag: | ||
commit: 3a1d936b7575b82197a1fea0632218dd07b1e65c | commit: 3a1d936b7575b82197a1fea0632218dd07b1e65c | ||
dist: . | dist: .git3a1d936.fc28 (snapshot date is computed once %{_sourcedir}/%{archivename}.%{archiveext} is available) | ||
=== Packaging a pre-release commit === | === Packaging a pre-release commit === |
Revision as of 13:13, 16 December 2017
Projects published on a “forge” can be packaged using the forgemeta macro.
Links
- FPC ticket
- fedora-rpm-macros RFE with the macro file
- packaging@lists.fedoraproject.org discussion
- pagure.io RFE to make pagure.io compatible with forgemeta
- GitLab CE RFE to remove GitLab quirks
- fedora-release RFE to make dist munging safe
Usage
A. The packager declares upstream-dependent metadata :
- forgeurl: the project URL on the target software hosting service
- version: the project release to package, if non nil (as Version: xxx)
- commit: the commit hash to package, if any
- tag: the project tag to package, if any
B. The packager calls %forgemeta. This macro will attempt to compute and set the following variables if they are not already set by the packager:
- forgesource: an URL that can be used as SourceX: value
- forgesetupargs: the correct arguments to pass to %setup for this source; used by %forgesetup and %forgeautosetup
- archivename: the source archive filename, without extentions
- archiveext: the source archive filename extensions, without leading dot
- archiveurl: the url that can be used to download the source archive, without renaming
- scm: the scm type, when packaging code snapshots (commits or tags)
If the macro is unable to parse the %{forgeurl} value the packager should set at least %{archivename} and %{archiveurl} before calling it.
The macro also accepts the following optional parameters:
- -u <url>: ignore %{forgeurl} even if it exists and use <url> instead; note that the macro will still end up setting <url> as %{forgeurl} if it manages to parse it
- -s: silently ignore problems in %{forgeurl}, use it if it can be parsed, ignore it otherwise
- -v : be verbose and print every variable the macro sets
- -i: Print some info about the state of variables the macro may use or set at the end of the processing
C. The packager uses the computed metadata as needed.
%{dist} modification for code snapshots
Most of the macro behavior is optional and safe. You can override the computed rpm variables values before or after the %forgemeta call, or ignore those variables altogether and don't use them in the rest of the spec file. There is one exception:
The macro will change the value of %{dist} in a non-reversible way, when packaging a code snapshot (commit or tag). Redefine %{dist} manually after the %forgemeta call if you don't like the result. Alternatively, don't use %forgemeta.
The root cause is that Fedora does not provide currently provide a placeholder prefix macro inside %{dist} that could be adjusted without redefining %{dist} as a whole.
Packaging examples
Packaging a release (except for GitLab)
%global forgeurl https://github.com/alecthomas/kingpin/ Version: 2.2.5 %forgemeta -i … Release: 1%{?dist} … URL: %{forgeurl} Source: %{forgesource} … %prep %forgesetup
$ rpmbuild -bs" output:
Forge-specific packaging variables forgeurl: https://github.com/alecthomas/kingpin forgesource: https://github.com/alecthomas/kingpin/archive/v2.2.5.tar.gz#/kingpin-2.2.5.tar.gz forgesetupargs: -n kingpin-2.2.5 Generic variables archivename: kingpin-2.2.5 archiveext: tar.gz archiveurl: https://github.com/alecthomas/kingpin/archive/v2.2.5.tar.gz scm: tag: commit: dist: .fc28 (snapshot date is computed once %{_sourcedir}/%{archivename}.%{archiveext} is available)
Packaging a release (GitLab)
%global forgeurl https://gitlab.example.com/foo/bar %global commit 2a810629566a2d0f0d4107df244e8828b9f7bd5c Version: 8.18.1 %forgemeta … Release: 1%{?dist} … URL: %{forgeurl} Source: %{forgesource} … %prep %forgesetup
Packaging a release tag
%global forgeurl https://github.com/apache/thrift/ Version: 0.10.0 %global tag %{version} %forgemeta -i … Release: 1%{?dist} … URL: %{forgeurl} Source: %{forgesource} … %prep %forgesetup
$ rpmbuild -bs" output:
Forge-specific packaging variables forgeurl: https://github.com/apache/thrift forgesource: https://github.com/apache/thrift/archive/0.10.0.tar.gz#/thrift-0.10.0.tar.gz forgesetupargs: -n thrift-0.10.0 Generic variables archivename: thrift-0.10.0 archiveext: tar.gz archiveurl: https://github.com/apache/thrift/archive/0.10.0.tar.gz scm: git tag: 0.10.0 commit: dist: .fc28 (snapshot date is computed once %{_sourcedir}/%{archivename}.%{archiveext} is available)
Packaging a commit
%global forgeurl https://code.googlesource.com/google-api-go-client/ %global commit 3a1d936b7575b82197a1fea0632218dd07b1e65c %forgemeta -i … Version: 0 Release: 0.1%{?dist} … URL: %{forgeurl} Source: %{forgesource} … %prep %forgesetup
$ rpmbuild -bs" output:
Forge-specific packaging variables forgeurl: https://code.googlesource.com/google-api-go-client forgesource: https://code.googlesource.com/google-api-go-client/+archive/3a1d936b7575b82197a1fea0632218dd07b1e65c.tar.gz#/google-api-go-client-3a1d936b7575b82197a1fea0632218dd07b1e65c.tar.gz forgesetupargs: -c Generic variables archivename: google-api-go-client-3a1d936b7575b82197a1fea0632218dd07b1e65c archiveext: tar.gz archiveurl: https://code.googlesource.com/google-api-go-client/+archive/3a1d936b7575b82197a1fea0632218dd07b1e65c.tar.gz scm: git tag: commit: 3a1d936b7575b82197a1fea0632218dd07b1e65c dist: .git3a1d936.fc28 (snapshot date is computed once %{_sourcedir}/%{archivename}.%{archiveext} is available)
Packaging a pre-release commit
%global forgeurl https://gitlab.example.com/foo/bar/ %global commit 51637bc0960002b811e1c0c7be8671cf9a1cc5be %forgemeta … Version: 3.2.6 Release: 0.1%{?dist} … URL: %{forgeurl} Source: %{forgesource} … %prep %forgesetup
Post-release packaging is similar. See also Packaging:Versioning#Release_and_post-release_versions.
Extending the macro
If the project you're packaging is published on a software publishing service forgemeta has no knowledge of, don't be sad, that's pretty easy to fix.
- note down the archive URLs you want to generate for versions, tags and commits
- locate the latest version of the forgemeta macro (it should be installed in /usr/lib/rpm/macros.d/macros.forge-srpm by fedora-rpm-macros)
- copy the definition block closest to your needs after the other definition blocks. It should look like :
if (forge == "vvv") then … end
or
if (string.match(forge, "vvv")) then … end
- change the "vvv" value
- adapt the initial normalization rule
forgeurl = string.match(forgeurl, "www")
- adapt the initial error message
print("%{error:"xxx URLs must match yyy!\\n}")
- set the hosting service defaults
safeset("archiveext", "tar.bz2") safeset("forgesetupargs", "-n %%{archivename}") if (commit ~= "") or (tag ~= "") then safeset("scm", "git") end
- scrap the variables you need from forgeurl using:
local myvariable = string.match(forgeurl, "zzz")
- use your variables to define archivename and archiveurl in the following if tag/commit/version block
- install the result in /usr/lib/rpm/macros.d/macros.forge-srpm
- test with forgemetacheck and rpmbuild -bs myspecfile.spec
- submit the enhancement for inclusion in fedora-rpm-macros once it works satisfactorily.
Benefits and limitations
- Benefits:
- Fedora packagers no longer need to know about the URL structure of well-known forges.
- Spec files are simpler, less error-prone, easier to maintain and audit.
- Forge URLs can be defined and fixed in a single place, without waiting for guidelines to percolate.
- The macros are mostly written in Lua, making them more verbose but easier to adjust and extend.
- The macros are written in Lua and can perform error handling.
- It is very easy to switch from commit to tag to version (or any combination of those).
- spectool just works© (in Fedora).
- scm snapshot date is exact and does not rely on a variable which may or may not have been updated.
- Almost all the computations are done in optional rpm variables that can be ignored by the packager if he does not need or does not like the result.
- Fedora forge know-how can be capitalized over time.
- The macro normalizes common spec constructs such as archivename.
- Limitations:
- The macro is not intended to be used in spec files that package multiple forge archives (it could probably be extended, is the added flexibility worth the complexity?).
- The macro needs to be updated when a forge changes its structure (but better changing one place than lots of spec files).
- New forges need to be added to the macro before it can be used with them.
- dist munging can not be easily reverted once forgemeta has been called. The solution would be to change fedora-release to define dist as %{?distprefix}.fcxx
- dist commit date computation relies on source files with the correct modification time (incorrect source file modification time → incorrect dist date).
- spectool may not work in older Fedora derivatives (but just use forgemetacheck and copy the source URL form its output).
- The macro highlights some historical rpm design mistakes: bad separation of upstream and package metadata, special magic variables. This is why version declaration occurs in an unusual (for rpm) order and requires a specific syntax.
Testing
Just drop in /usr/lib/rpm/macros.d/ the file proposed here for inclusion in fedora-rpm-macros, and play with the result. The file name must be prefixed with “macros.”.