Skip to content

Commit 920bddd

Browse files
Maintenance to Fedora CI infrastructure and upgrade to nl::json 3.12.0 (#363)
* Simplify fedora CI * Update pre-commits * Run pre-commit * Disable tests that cannot be built * Skip failing tests * Library was renamed * Disable aarch64 for now * copy-paste leftover * Disable testing-farm tests for now * Upgraded to nlohmann_json 3.12.0 * Moved json-schema-validate under test --------- Co-authored-by: Johan Mabille <[email protected]>
1 parent 40af3ec commit 920bddd

15 files changed

+86
-126
lines changed

.distro/.gitignore

Lines changed: 0 additions & 4 deletions
This file was deleted.

.distro/json-schema-validator.rpmlintrc

Whitespace-only changes.

.distro/json-schema-validator.spec

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,10 @@ Json schema validator development files for C++ projects using nlohmann/json
4949
%files
5050
%doc README.md
5151
%license LICENSE
52-
%{_libdir}/libnlohmann_json_validator.so.*
52+
%{_libdir}/libnlohmann_json_schema_validator.so.*
5353

5454
%files devel
55-
%{_libdir}/libnlohmann_json_validator.so
55+
%{_libdir}/libnlohmann_json_schema_validator.so
5656
%{_includedir}/nlohmann/json-schema.hpp
5757
%{_libdir}/cmake/nlohmann_json_schema_validator
5858

.distro/packit.toml

Lines changed: 0 additions & 3 deletions
This file was deleted.

.distro/plans/main.fmf.dist-git

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,10 @@
11
discover:
22
how: fmf
3-
dist-git-source: true
4-
path: .distro
3+
path: .
4+
5+
adjust+:
6+
# Cannot use initiator: fedora-ci reliably yet
7+
when: initiator is not defined or initiator != packit
8+
discover+:
9+
dist-git-source: true
10+
dist-git-extract: json-schema-validator-*/

.distro/plans/rpmlint.fmf

Lines changed: 0 additions & 14 deletions
This file was deleted.

.distro/tests/rpmlint.fmf

Lines changed: 0 additions & 13 deletions
This file was deleted.

.github/workflows/test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ jobs:
3030
fail-fast: false
3131
matrix:
3232
toolchain: [ gcc, llvm, intel ]
33-
json_version: [ v3.11.2, v3.8.0 ]
33+
json_version: [ develop, v3.12.0, v3.8.0 ]
3434
experimental: [ false ]
3535
include:
3636
- toolchain: llvm

.packit.yaml

Lines changed: 62 additions & 73 deletions
Original file line numberDiff line numberDiff line change
@@ -1,79 +1,68 @@
1-
specfile_path: .distro/json-schema-validator.spec
21
files_to_sync:
3-
- src: .distro/json-schema-validator.spec
4-
dest: json-schema-validator.spec
5-
- .packit.yaml
6-
- src: .distro/json-schema-validator.rpmlintrc
7-
dest: json-schema-validator.rpmlintrc
8-
# tmt setup
9-
- src: .distro/.fmf/
10-
dest: .fmf/
11-
- src: .distro/plans/
12-
dest: plans/
13-
filters:
14-
- "- .distro/plans/main.fmf.dist-git"
15-
- "- .distro/plans/rpmlint.fmf"
16-
- src: .distro/plans/main.fmf.dist-git
17-
dest: plans/main.fmf
2+
- src: .distro/
3+
dest: ./
4+
delete: true
5+
filters:
6+
- "protect .git*"
7+
- "protect sources"
8+
- "protect changelog"
9+
- "protect gating.yaml"
10+
# Temporary workaround until
11+
# https://github.com/packit/packit/pull/2573
12+
- "- json-schema-validator.spec"
13+
- .packit.yaml
14+
1815
upstream_package_name: json-schema-validator
16+
specfile_path: .distro/json-schema-validator.spec
1917
downstream_package_name: json-schema-validator
2018
update_release: false
21-
upstream_tag_template: v{version}
19+
20+
targets: &targets
21+
- fedora-all-x86_64
22+
# TODO: aarch64 is failing at test
23+
# JSON-Suite::Optional::Format::idn-email
24+
# - fedora-all-aarch64
25+
26+
_:
27+
# Job templates
28+
- &build-in-packit
29+
job: copr_build
30+
- &build-in-lecris
31+
<<: *build-in-packit
32+
owner: "@scikit-build"
33+
- &tests
34+
job: tests
35+
fmf_path: .distro
36+
identifier: downstream
2237

2338
jobs:
24-
- job: copr_build
25-
trigger: pull_request
26-
owner: lecris
27-
project: json-schema-validator
28-
update_release: true
29-
release_suffix: "{PACKIT_RPMSPEC_RELEASE}"
30-
targets:
31-
- fedora-development
32-
- job: tests
33-
trigger: pull_request
34-
targets:
35-
- fedora-development
36-
fmf_path: .distro
37-
- job: copr_build
38-
trigger: commit
39-
branch: main
40-
owner: lecris
41-
project: nightly
42-
# TODO: Remove when upstream issue is resolved
43-
# https://github.com/packit/packit/issues/1924
44-
additional_repos:
45-
- copr://@scikit-build/release
46-
targets:
47-
- fedora-development-x86_64
48-
- fedora-latest-x86_64
49-
- fedora-development-aarch64
50-
- fedora-latest-aarch64
51-
- job: copr_build
52-
trigger: release
53-
owner: lecris
54-
project: release
55-
targets:
56-
- fedora-development-x86_64
57-
- fedora-latest-x86_64
58-
- fedora-development-aarch64
59-
- fedora-latest-aarch64
60-
- job: tests
61-
trigger: commit
62-
branch: main
63-
targets:
64-
- fedora-development
65-
- fedora-latest
66-
fmf_path: .distro
67-
- job: propose_downstream
68-
trigger: release
69-
dist_git_branches:
70-
- fedora-development
71-
- fedora-latest
72-
- job: koji_build
73-
trigger: commit
74-
dist_git_branches:
75-
- fedora-all
76-
- job: bodhi_update
77-
trigger: commit
78-
dist_git_branches:
79-
- fedora-branched
39+
# Upstream jobs
40+
- <<: *build-in-lecris
41+
trigger: release
42+
project: release
43+
# - <<: *tests
44+
# trigger: release
45+
- <<: *build-in-lecris
46+
trigger: commit
47+
branch: main
48+
project: nightly
49+
# - <<: *tests
50+
# trigger: commit
51+
# branch: main
52+
- <<: *build-in-packit
53+
trigger: pull_request
54+
# - <<: *tests
55+
# trigger: pull_request
56+
# Downstream jobs
57+
- job: propose_downstream
58+
trigger: release
59+
dist_git_branches:
60+
- fedora-rawhide
61+
- job: koji_build
62+
trigger: commit
63+
dist_git_branches:
64+
- fedora-all
65+
- job: bodhi_update
66+
trigger: commit
67+
dist_git_branches:
68+
- fedora-branched

.pre-commit-config.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
repos:
22
- repo: https://github.com/Takishima/cmake-pre-commit-hooks
3-
rev: v1.8.1
3+
rev: v1.9.6
44
hooks:
55
- id: clang-format
66
args:
@@ -15,19 +15,19 @@ repos:
1515
- 'pre-commit'
1616
stages: [ manual ]
1717
- repo: https://github.com/pre-commit/pre-commit-hooks
18-
rev: v4.4.0
18+
rev: v6.0.0
1919
hooks:
2020
- id: trailing-whitespace
2121
- id: end-of-file-fixer
2222
- id: check-yaml
2323
- repo: https://github.com/executablebooks/mdformat
24-
rev: 0.7.16
24+
rev: 0.7.22
2525
hooks:
2626
- id: mdformat
2727
additional_dependencies:
2828
- mdformat-gfm
2929
- mdformat-tables
3030
- repo: https://github.com/python-jsonschema/check-jsonschema
31-
rev: 0.23.0
31+
rev: 0.34.0
3232
hooks:
3333
- id: check-github-workflows

0 commit comments

Comments
 (0)