- gerrit-log - lists.osmocom.org

Change in libosmo-gprs[master]: configure: generate libosmo-gprs.spec from libosmo-gprs.spec.in
by fixeria 17 Aug '22

17 Aug '22

fixeria has uploaded a new patch set (#2). ( https://gerrit.osmocom.org/c/libosmo-gprs/+/29148 ) Change subject: configure: generate libosmo-gprs.spec from libosmo-gprs.spec.in ...................................................................... configure: generate libosmo-gprs.spec from libosmo-gprs.spec.in Change-Id: Ie15c136d9f1919da0217b17d1afa9268f7ceaab1 --- M .gitignore M Makefile.am M configure.ac 3 files changed, 5 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/48/29148/2 -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/29148 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: Ie15c136d9f1919da0217b17d1afa9268f7ceaab1 Gerrit-Change-Number: 29148 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newpatchset

1 0

Change in libosmo-gprs[master]: contrib/libosmo-gprs.spec.in: properly define RPM packages
by fixeria 17 Aug '22

17 Aug '22

Attention is currently required from: osmith. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/29114 ) Change subject: contrib/libosmo-gprs.spec.in: properly define RPM packages ...................................................................... Patch Set 3: (2 comments) File contrib/libosmo-gprs.spec.in: https://gerrit.osmocom.org/c/libosmo-gprs/+/29114/comment/400a46a3_4623629d PS2, Line 46: Group: System/Libraries > Why changing tabs to spaces? Nevermind. I see other osmo-projects use spaces in their *.spec files, so let's be consistent. https://gerrit.osmocom.org/c/libosmo-gprs/+/29114/comment/d3961939_2283e191 PS2, Line 56: Requires: libosmo-csn1-devel = %{version} > Why changing tabs to spaces? See above. -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/29114 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: I9be45c43ba3e342e1dba4b60561880c00841f549 Gerrit-Change-Number: 29114 Gerrit-PatchSet: 3 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Wed, 17 Aug 2022 22:29:23 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: comment

1 0

Change in libosmo-gprs[master]: contrib/libosmo-gprs.spec.in: properly define RPM packages
by fixeria 17 Aug '22

17 Aug '22

Attention is currently required from: osmith. Hello osmith, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/libosmo-gprs/+/29114 to look at the new patch set (#3). Change subject: contrib/libosmo-gprs.spec.in: properly define RPM packages ...................................................................... contrib/libosmo-gprs.spec.in: properly define RPM packages Co-Authored-By: Oliver Smith <osmith(a)sysmocom.de> Change-Id: I9be45c43ba3e342e1dba4b60561880c00841f549 --- M contrib/libosmo-gprs.spec.in 1 file changed, 77 insertions(+), 8 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/14/29114/3 -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/29114 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: I9be45c43ba3e342e1dba4b60561880c00841f549 Gerrit-Change-Number: 29114 Gerrit-PatchSet: 3 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newpatchset

1 0

Change in libosmo-gprs[master]: configure: generate libosmo-gprs.spec from libosmo-gprs.spec.in
by fixeria 17 Aug '22

17 Aug '22

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/29148 ) Change subject: configure: generate libosmo-gprs.spec from libosmo-gprs.spec.in ...................................................................... configure: generate libosmo-gprs.spec from libosmo-gprs.spec.in Change-Id: Ie15c136d9f1919da0217b17d1afa9268f7ceaab1 --- M .gitignore M Makefile.am M configure.ac 3 files changed, 5 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/48/29148/1 diff --git a/.gitignore b/.gitignore index 3935e05..6efb54d 100644 --- a/.gitignore +++ b/.gitignore @@ -54,3 +54,5 @@ tags /deps + +contrib/libosmo-gprs.spec diff --git a/Makefile.am b/Makefile.am index 79aee53..ebaaf29 100644 --- a/Makefile.am +++ b/Makefile.am @@ -21,6 +21,7 @@ BUILT_SOURCES = $(top_srcdir)/.version EXTRA_DIST = \ .version \ + contrib/libosmo-gprs.spec.in \ git-version-gen \ debian \ $(NULL) diff --git a/configure.ac b/configure.ac index 15974ee..f0d0401 100644 --- a/configure.ac +++ b/configure.ac @@ -87,5 +87,6 @@ src/rlcmac/Makefile tests/Makefile tests/rlcmac/Makefile - Makefile]) + Makefile + libosmo-gprs.spec]) AC_OUTPUT -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/29148 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: Ie15c136d9f1919da0217b17d1afa9268f7ceaab1 Gerrit-Change-Number: 29148 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

Change in libosmo-gprs[master]: contrib/libosmo-gprs.spec.in: properly define RPM packages
by fixeria 17 Aug '22

17 Aug '22

Attention is currently required from: osmith. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/29114 ) Change subject: contrib/libosmo-gprs.spec.in: properly define RPM packages ...................................................................... Patch Set 2: (5 comments) File contrib/libosmo-gprs.spec.in: https://gerrit.osmocom.org/c/libosmo-gprs/+/29114/comment/fba3c685_99a5c545 PS2, Line 44: Osmocom GPRS RLC/MAC library Good catch, thanks for fixing this. https://gerrit.osmocom.org/c/libosmo-gprs/+/29114/comment/b9a33ba7_6b1606ac PS2, Line 46: Group: System/Libraries Why changing tabs to spaces? https://gerrit.osmocom.org/c/libosmo-gprs/+/29114/comment/c3ba2519_8ddc5e30 PS2, Line 56: Requires: libosmo-csn1-devel = %{version} Why changing tabs to spaces? https://gerrit.osmocom.org/c/libosmo-gprs/+/29114/comment/aa391254_2a8d29b8 PS2, Line 72: --includedir="%_includedir/%name" Both --disable-static and --enable-shared are implicit defaults. I guess missing --includedir was the problem? https://gerrit.osmocom.org/c/libosmo-gprs/+/29114/comment/18230766_54fff924 PS2, Line 77: find "%buildroot/%_libdir" -type f -name "*.la" -delete Interesting. This is something I also noticed while looking at *.spec files in other Osmocom projects: *-dev packages for Debian usually include lib*.{a,la,so} files, while *-devel packages for RPM based distributions do not include lib*.{a,la}. Why do we remove *.la files here if we don't list them below anyway? -- To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/29114 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-gprs Gerrit-Branch: master Gerrit-Change-Id: I9be45c43ba3e342e1dba4b60561880c00841f549 Gerrit-Change-Number: 29114 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Wed, 17 Aug 2022 21:57:56 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in osmo-ci[master]: osmocom-list-commits: use proper git clone urls
by fixeria 17 Aug '22

17 Aug '22

Attention is currently required from: osmith, neels. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29119 ) Change subject: osmocom-list-commits: use proper git clone urls ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: > This is really weird. It's available at git.osmocom.org over git://, but not over https://. I created https://osmocom.org/issues/5650 to track this. -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29119 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I2acb9880be57411f11805a2195076b514156aaf3 Gerrit-Change-Number: 29119 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Comment-Date: Wed, 17 Aug 2022 20:56:03 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: comment

1 0

Change in osmo-upf[master]: update git URLs (git -> https; gitea)
by fixeria 17 Aug '22

17 Aug '22

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-upf/+/29147 ) Change subject: update git URLs (git -> https; gitea) ...................................................................... update git URLs (git -> https; gitea) Change-Id: I7e0190b36861710d8cc8dd0cfd50d3dbe301ede5 --- M README.md M configure.ac M debian/control M debian/copyright 4 files changed, 6 insertions(+), 6 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-upf refs/changes/47/29147/1 diff --git a/README.md b/README.md index 5e61811..73a6604 100644 --- a/README.md +++ b/README.md @@ -12,9 +12,9 @@ You can clone from the official osmo-upf.git repository using - git clone git://git.osmocom.org/osmo-upf.git + git clone https://gitea.osmocom.org/cellular-infrastructure/osmo-upf -There is a cgit interface at https://git.osmocom.org/osmo-upf/ +There is a web interface at https://gitea.osmocom.org/cellular-infrastructure/osmo-upf. To submit patches, see "Contributing" below. diff --git a/configure.ac b/configure.ac index 18e8d6b..a900f10 100644 --- a/configure.ac +++ b/configure.ac @@ -125,7 +125,7 @@ fi AC_CHECK_PROG(OSMOTESTEXT_CHECK,osmotestvty.py,yes) if test "x$OSMOTESTEXT_CHECK" != "xyes" ; then - AC_MSG_ERROR([Please install git://osmocom.org/python/osmo-python-tests to run the VTY/CTRL tests.]) + AC_MSG_ERROR([Please install https://gitea.osmocom.org/cellular-infrastructure/osmo-python-tests to run the VTY/CTRL tests.]) fi fi AC_MSG_CHECKING([whether to enable VTY/CTRL tests]) diff --git a/debian/control b/debian/control index cabcd19..9b7d40a 100644 --- a/debian/control +++ b/debian/control @@ -16,8 +16,8 @@ libosmocore-dev (>= 1.6.0), osmo-gsm-manuals-dev (>= 1.2.0) Standards-Version: 3.9.8 -Vcs-Git: git://git.osmocom.org/osmo-upf.git -Vcs-Browser: https://git.osmocom.org/osmo-upf/ +Vcs-Git: https://gitea.osmocom.org/cellular-infrastructure/osmo-upf +Vcs-Browser: https://gitea.osmocom.org/cellular-infrastructure/osmo-upf Homepage: https://projects.osmocom.org/projects/osmo-upf Package: osmo-upf diff --git a/debian/copyright b/debian/copyright index 84c249f..b72a15d 100644 --- a/debian/copyright +++ b/debian/copyright @@ -1,6 +1,6 @@ Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/ Upstream-Name: osmo-upf -Source: git://git.osmocom.org/osmo-upf +Source: https://gitea.osmocom.org/cellular-infrastructure/osmo-upf Files: * Copyright: 2021-2022 sysmocom - s.f.m.c. GmbH <info(a)sysmocom.de> -- To view, visit https://gerrit.osmocom.org/c/osmo-upf/+/29147 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-upf Gerrit-Branch: master Gerrit-Change-Id: I7e0190b36861710d8cc8dd0cfd50d3dbe301ede5 Gerrit-Change-Number: 29147 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

Change in osmo-upf[master]: Fix missing dash in Redmine project URL
by fixeria 17 Aug '22

17 Aug '22

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-upf/+/29146 ) Change subject: Fix missing dash in Redmine project URL ...................................................................... Fix missing dash in Redmine project URL Change-Id: I2e3fd2ec734799098fed1ea361b8236746c97812 --- M README.md M contrib/osmo-upf.spec.in 2 files changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-upf refs/changes/46/29146/1 diff --git a/README.md b/README.md index 8d00fe3..5e61811 100644 --- a/README.md +++ b/README.md @@ -5,7 +5,7 @@ -------- The official homepage of the project is -https://osmocom.org/projects/osmoupf/wiki +https://osmocom.org/projects/osmo-upf/wiki GIT Repository -------------- diff --git a/contrib/osmo-upf.spec.in b/contrib/osmo-upf.spec.in index 78e8aae..a65b7ef 100644 --- a/contrib/osmo-upf.spec.in +++ b/contrib/osmo-upf.spec.in @@ -22,7 +22,7 @@ Summary: OsmoUPF: Osmocom User Plane Function License: AGPL-3.0-or-later AND GPL-2.0-or-later Group: Hardware/Mobile -URL: https://osmocom.org/projects/osmoupf +URL: https://osmocom.org/projects/osmo-upf Source: %{name}-%{version}.tar.xz BuildRequires: autoconf-archive BuildRequires: automake >= 1.9 -- To view, visit https://gerrit.osmocom.org/c/osmo-upf/+/29146 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-upf Gerrit-Branch: master Gerrit-Change-Id: I2e3fd2ec734799098fed1ea361b8236746c97812 Gerrit-Change-Number: 29146 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

Change in osmo-ci[master]: osmocom-list-commits: use proper git clone urls
by fixeria 17 Aug '22

17 Aug '22

Attention is currently required from: osmith, neels. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29119 ) Change subject: osmocom-list-commits: use proper git clone urls ...................................................................... Patch Set 1: (1 comment) Patchset: PS1: > Hmm, so AFAIU, osmo-upf is not available over git.osmocom.org, but only via gerrit.osmocom. […] This is really weird. It's available at git.osmocom.org over git://, but not over https://. -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29119 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I2acb9880be57411f11805a2195076b514156aaf3 Gerrit-Change-Number: 29119 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Comment-Date: Wed, 17 Aug 2022 20:38:02 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: comment

1 0

Change in osmo-ci[master]: osmocom-list-commits: use proper git clone urls
by fixeria 17 Aug '22

17 Aug '22

Attention is currently required from: osmith. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/29119 ) Change subject: osmocom-list-commits: use proper git clone urls ...................................................................... Patch Set 1: Code-Review+1 (1 comment) Patchset: PS1: Hmm, so AFAIU, osmo-upf is not available over git.osmocom.org, but only via gerrit.osmocom.org? -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29119 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: I2acb9880be57411f11805a2195076b514156aaf3 Gerrit-Change-Number: 29119 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Wed, 17 Aug 2022 20:35:53 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

2026

2025

2024

2023

2022

gerrit-log