PR#8: 0.17.6 upstream release - rpms/python-scikit-build

rpms / python-scikit-build

#8 0.17.6 upstream release

Merged a year ago by lecris. Opened a year ago by lecris.

rpms/ lecris/python-scikit-build 0.17.6 into rawhide

Packit • a year ago

0aecd8e

.fmf/version

file added

		`@@ -0,0 +1,1 @@`
		`+ 1`

.gitignore

file modified

		`@@ -1,3 +1,4 @@`
		`/scikit-build-0.16.6.tar.gz`
		`/scikit-build-0.16.7.tar.gz`
		`/scikit_build-0.17.1.tar.gz`
		`+ /scikit_build-0.17.6.tar.gz`

.packit.yaml

file added

+81

		`@@ -0,0 +1,81 @@`
		`+ # See the documentation for more information:`
		`+ # https://packit.dev/docs/configuration/`
		`+`
		`+ specfile_path: .distro/python-scikit-build.spec`
		`+`
		`+ files_to_sync:`
		`+ - src: .distro/python-scikit-build.spec`
		`+ dest: python-scikit-build.spec`
		`+ - .packit.yaml`
		`+ - src: .distro/python-scikit-build.rpmlintrc`
		`+ dest: python-scikit-build.rpmlintrc`
		`+ # tmt setup`
		`+ - src: .distro/.fmf/`
		`+ dest: .fmf/`
		`+ - src: .distro/plans/`
		`+ dest: plans/`
		`+ filters:`
		`+ - "- .distro/plans/main.fmf.dist-git"`
		`+ - "- .distro/plans/rpmlint.fmf"`
		`+ - src: .distro/plans/main.fmf.dist-git`
		`+ dest: plans/main.fmf`
		`+ upstream_package_name: scikit-build`
		`+ downstream_package_name: python-scikit-build`
		`+ update_release: false`
		`+`
		`+ jobs:`
		`+ - job: copr_build`
		`+ trigger: pull_request`
		`+ owner: "@scikit-build"`
		`+ project: scikit-build`
		`+ update_release: true`
		`+ release_suffix: "{PACKIT_RPMSPEC_RELEASE}"`
		`+ targets:`
		`+ - fedora-development`
		`+ - job: tests`
		`+ trigger: pull_request`
		`+ targets:`
		`+ - fedora-development`
		`+ fmf_path: .distro`
		`+ - job: copr_build`
		`+ trigger: commit`
		`+ branch: main`
		`+ owner: "@scikit-build"`
		`+ project: nightly`
		`+ targets:`
		`+ - fedora-development-x86_64`
		`+ - fedora-latest-x86_64`
		`+ - fedora-development-aarch64`
		`+ - fedora-latest-aarch64`
		`+ - job: tests`
		`+ trigger: commit`
		`+ branch: main`
		`+ targets:`
		`+ - fedora-development`
		`+ - fedora-latest`
		`+ fmf_path: .distro`
		`+ - job: copr_build`
		`+ trigger: release`
		`+ owner: "@scikit-build"`
		`+ project: release`
		`+ targets:`
		`+ - fedora-development-x86_64`
		`+ - fedora-latest-x86_64`
		`+ - fedora-development-aarch64`
		`+ - fedora-latest-aarch64`
		`+ - job: propose_downstream`
		`+ trigger: release`
		`+ dist_git_branches:`
		`+ # TODO: Switch to fedora-development and fedora-latest`
		`+ # There is an issue that the commits diverge on different PRs. In the meantime will create PRs on branched fedora`
		`+ # manually`
		`+ # https://github.com/packit/packit/issues/1724`
		`+ - fedora-rawhide`
		`+ - job: koji_build`
		`+ trigger: commit`
		`+ dist_git_branches:`
		`+ - fedora-all`
		`+ - job: bodhi_update`
		`+ trigger: commit`
		`+ dist_git_branches:`
		`+ - fedora-branched`

0001-tests-fix-match-for-get_map-symbol.patch

file removed

-27

		`@@ -1,27 +0,0 @@`
		`- From 04f41fbf465c78b3717f88100cc8f4a9229085ea Mon Sep 17 00:00:00 2001`
		`- From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>`
		`- Date: Sat, 27 May 2023 19:22:23 +0200`
		`- Subject: [PATCH] tests: fix match for get_map symbol`
		`-`
		`- Wrong line could be matched by mistake, e.g.`
		`- 45: 0000000000010160 0 NOTYPE LOCAL DEFAULT 10 0000001a.plt_call.get_map`
		`- ---`
		`- tests/test_issue668_symbol_visibility.py \| 2 +-`
		`- 1 file changed, 1 insertion(+), 1 deletion(-)`
		`-`
		`- diff --git a/tests/test_issue668_symbol_visibility.py b/tests/test_issue668_symbol_visibility.py`
		`- index e52912be79..f81b2bd60d 100644`
		`- --- a/tests/test_issue668_symbol_visibility.py`
		`- +++ b/tests/test_issue668_symbol_visibility.py`
		`- @@ -56,7 +56,7 @@ def test_symbol_visibility(skip_override):`
		`- # Looking for entries associated with get_map`
		`- # ex. 62: 0000000000001260 164 FUNC GLOBAL DEFAULT 14 get_map\n`
		`- # NOTE: We want to ignore get_map::id_to_resourse entries`
		`- - if "get_map" in line and "get_map::" not in line:`
		`- + if "get_map" in line.split():`
		`- print(line)`
		`- if skip_override == "ON":`
		`- assert "GLOBAL" in line`
		`- --`
		`- 2.40.1`
		`-`

README.packit

file added

		`@@ -0,0 +1,3 @@`
		`+ This repository is maintained by packit.`
		`+ https://packit.dev/`
		`+ The file was generated using packit 0.76.0.`

plans/main.fmf

file added

		`@@ -0,0 +1,3 @@`
		`+ discover:`
		`+ how: fmf`
		`+ dist-git-source: true`

plans/smoke.fmf

file added

		`@@ -0,0 +1,7 @@`
		`+ summary:`
		`+ Basic smoke tests`
		`+ discover+:`
		`+ how: fmf`
		`+ filter: "tag: smoke"`
		`+ execute:`
		`+ how: tmt`

python-scikit-build.rpmlintrc

file added

empty or binary file added

python-scikit-build.spec

file modified

+6 -15

		`@@ -1,8 +1,6 @@`
		`- %global pypi_name scikit_build`
		`-`
		`Name: python-scikit-build`
		`- Version: 0.17.1`
		`- Release: %{autorelease}`
		`+ Version: 0.17.6`
		`+ Release: %autorelease`
		`Summary: Improved build system generator for Python C/C++/Fortran/Cython extensions`

		`# This project is mainly MIT but LICENSE also mentions some code`
		`@@ -10,9 +8,7 @@`
		`# All bundled(cmake()) files listed are Apache-2.0 licensed.`
		`License: MIT AND BSD-2-Clause-Views AND Apache-2.0`
		`URL: https://github.com/scikit-build/scikit-build`
		`- Source0: %{pypi_source %{pypi_name}}`
		`-`
		`- Patch: 0001-tests-fix-match-for-get_map-symbol.patch`
		`+ Source: %{pypi_source scikit_build}`

		`BuildArch: noarch`
		`BuildRequires: python3-devel`
		`@@ -54,7 +50,7 @@`


		`%prep`
		`- %autosetup -p1 -n %{pypi_name}-%{version}`
		`+ %autosetup -p1 -n scikit_build-%{version}`


		`%generate_buildrequires`
		`@@ -75,13 +71,8 @@`
		`# so we clean them.`
		`export CFLAGS=' '`
		`export CXXFLAGS=' '`
		`- OPTIONS=(`
		`- # pep518 tests are disabled because they require internet`
		`- -k "not pep518"`
		`-`
		`- -m "not deprecated and not nosetuptoolsscm"`
		`- )`
		`- %pytest -v "${OPTIONS[@]}"`
		`+ # isolated tests are disabled because they require internet`
		`+ %pytest -v -m "not isolated and not deprecated and not nosetuptoolsscm"`


		`%files -n python3-scikit-build -f %{pyproject_files}`

sources

file modified

+1 -1

		`@@ -1,1 +1,1 @@`
		`- SHA512 (scikit_build-0.17.1.tar.gz) = 617a68d92787c576f1c3fc9c24c8cc90e1fe976a853f3ae81442e1a99a20e1f4049d7ffac967c2248bf172f07a3e2092fc28b8d558b35da628a58cd3d9d16ae3`
		`+ SHA512 (scikit_build-0.17.6.tar.gz) = 92b97146f40d8222bd8415ef8439497d5075b72e5bafc40aba0a3e2911d269a118f2f7d41f468f7add8949f550b1fd7d5a9113d249e42dfac431108182ca9198`

lecris commented a year ago

[Fixed packit PR]

Upstream tag: 0.17.6
Upstream commit: 4eef05f0

lecris commented a year ago

@thrnciar Can you do your magic on this PR? And review this one?

Metadata Update from @lecris:
- Request assigned

a year ago

Metadata Update from @lecris:
- Request assigned

a year ago

rebased onto 0f43127

a year ago

rebased onto 0aecd8e

a year ago

zuul commented a year ago

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/2b9da0770d344e688e7a16fd48298cfa

check-for-arches : SUCCESS in 2m 55s
rpm-scratch-build : SUCCESS in 12m 37s
rpm-linter : SUCCESS in 7m 37s (non-voting)
rpm-rpminspect : SUCCESS in 10m 45s (non-voting)
check-for-sti-tests : SUCCESS in 17s
check-for-fmf-tests : SUCCESS in 18s
rpm-install-test : SUCCESS in 1m 27s
rpm-tmt-test : SUCCESS in 7m 40s
Skipped 5 jobs

thrnciar commented a year ago

I am sorry, I triggered the magic and forgot to reply. The impact check looks good, there is only one depending package and it builds.

https://copr.fedorainfracloud.org/coprs/thrnciar/python-scikit-build-4b172772/builds/

thrnciar commented a year ago

[recheck] [citest]

lecris commented a year ago

Can you also check the other PR for F38? (Ignore the first failing zuul, that is from me trying to PR into f38, but the commits have diverged)

PS: are you present on matrix/irc so I can ping you there next time?

Edited a year ago by lecris

zuul commented a year ago

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/91e425eb71954313b074225ddbce558a

check-for-arches : SUCCESS in 2m 13s
rpm-scratch-build : SUCCESS in 14m 33s
rpm-linter : SUCCESS in 10m 56s (non-voting)
rpm-rpminspect : SUCCESS in 14m 32s (non-voting)
check-for-sti-tests : SUCCESS in 17s
check-for-fmf-tests : SUCCESS in 17s
rpm-install-test : SUCCESS in 1m 23s
rpm-tmt-test : SUCCESS in 9m 32s
Skipped 5 jobs

thrnciar commented a year ago

I am trying to be present at #python channel on fedora's matrix, although it kicks me off the room once in a while if I am inactive. Feel free to ping me there or via email thrnciar AT redhat DOT com.

Pull-Request has been merged by lecris

a year ago