- gerrit-log - lists.osmocom.org

Change in osmo-ci[master]: Revert "remove linter jobs for osmo-e1d and osmo-e1-firmware"

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/26944 ) Change subject: Revert "remove linter jobs for osmo-e1d and osmo-e1-firmware" ...................................................................... Revert "remove linter jobs for osmo-e1d and osmo-e1-firmware" Enable the linter again, as we decided to use a repo-specific checkpatch config in these. This reverts commit 2c6aa82c9a8a81addd805fe211b048076e01f8c5. Related: OS#5399 Change-Id: Ib79a20658bfae64d9b718e53e6a9393708a4e694 --- M jobs/gerrit-lint.yml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/jobs/gerrit-lint.yml b/jobs/gerrit-lint.yml index b98a041..45db681 100644 --- a/jobs/gerrit-lint.yml +++ b/jobs/gerrit-lint.yml @@ -73,7 +73,9 @@ - simtrace2 - osmo-asf4-dfu - osmo-ccid-firmware + - osmo-e1d - osmo-cbc + - osmo-e1-recorder - gapk - osmo-uecups - osmo-el2tpd -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/26944 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ib79a20658bfae64d9b718e53e6a9393708a4e694 Gerrit-Change-Number: 26944 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged

3 years, 6 months

1
0
0 0

Change in osmo-ci[master]: Revert "remove linter jobs for osmo-e1d and osmo-e1-firmware"

by laforge

Attention is currently required from: osmith. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/26944 ) Change subject: Revert "remove linter jobs for osmo-e1d and osmo-e1-firmware" ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/26944 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ib79a20658bfae64d9b718e53e6a9393708a4e694 Gerrit-Change-Number: 26944 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Tue, 18 Jan 2022 09:07:25 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 6 months

1
0
0 0

Change in osmo-e1d[master]: checkpatch.conf: new file

by laforge

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-e1d/+/26945 ) Change subject: checkpatch.conf: new file ...................................................................... checkpatch.conf: new file Related: OS#5399 Change-Id: I4bd109009cbe16f313e19ab78bd5fbcb13b0f656 --- A .checkpatch.conf 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/.checkpatch.conf b/.checkpatch.conf new file mode 100644 index 0000000..20c0832 --- /dev/null +++ b/.checkpatch.conf @@ -0,0 +1,2 @@ +--ignore OPEN_BRACE +--ignore SPACING -- To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/26945 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1d Gerrit-Branch: master Gerrit-Change-Id: I4bd109009cbe16f313e19ab78bd5fbcb13b0f656 Gerrit-Change-Number: 26945 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged

3 years, 6 months

1
0
0 0

Change in osmo-e1d[master]: checkpatch.conf: new file

by laforge

Attention is currently required from: osmith. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-e1d/+/26945 ) Change subject: checkpatch.conf: new file ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/26945 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1d Gerrit-Branch: master Gerrit-Change-Id: I4bd109009cbe16f313e19ab78bd5fbcb13b0f656 Gerrit-Change-Number: 26945 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Tue, 18 Jan 2022 09:07:07 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 6 months

1
0
0 0

Change in osmo-e1-recorder[master]: checkpatch.conf: new file

by laforge

Attention is currently required from: osmith. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-e1-recorder/+/26946 ) Change subject: checkpatch.conf: new file ...................................................................... Patch Set 1: Code-Review-2 (1 comment) Patchset: PS1: this was actually an error on my side. osmo-e1-recorder is a normal osmocom project with no exceptions. -- To view, visit https://gerrit.osmocom.org/c/osmo-e1-recorder/+/26946 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1-recorder Gerrit-Branch: master Gerrit-Change-Id: I8b6f5f0b1dd980003a4ae2feaee78fc62debbc9c Gerrit-Change-Number: 26946 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Tue, 18 Jan 2022 09:06:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 6 months

1
0
0 0

Change in osmo-e1-recorder[master]: checkpatch.conf: new file

by laforge

Attention is currently required from: osmith. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-e1-recorder/+/26946 ) Change subject: checkpatch.conf: new file ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-e1-recorder/+/26946 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1-recorder Gerrit-Branch: master Gerrit-Change-Id: I8b6f5f0b1dd980003a4ae2feaee78fc62debbc9c Gerrit-Change-Number: 26946 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: osmith <osmith(a)sysmocom.de> Gerrit-Comment-Date: Tue, 18 Jan 2022 09:06:20 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

3 years, 6 months

1
0
0 0

Change in osmo-e1-recorder[master]: checkpatch.conf: new file

by osmith

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-e1-recorder/+/26946 ) Change subject: checkpatch.conf: new file ...................................................................... checkpatch.conf: new file Related: OS#5399 Change-Id: I8b6f5f0b1dd980003a4ae2feaee78fc62debbc9c --- A .checkpatch.conf 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-e1-recorder refs/changes/46/26946/1 diff --git a/.checkpatch.conf b/.checkpatch.conf new file mode 100644 index 0000000..20c0832 --- /dev/null +++ b/.checkpatch.conf @@ -0,0 +1,2 @@ +--ignore OPEN_BRACE +--ignore SPACING -- To view, visit https://gerrit.osmocom.org/c/osmo-e1-recorder/+/26946 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1-recorder Gerrit-Branch: master Gerrit-Change-Id: I8b6f5f0b1dd980003a4ae2feaee78fc62debbc9c Gerrit-Change-Number: 26946 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

3 years, 6 months

1
0
0 0

Change in osmo-e1d[master]: checkpatch.conf: new file

by osmith

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-e1d/+/26945 ) Change subject: checkpatch.conf: new file ...................................................................... checkpatch.conf: new file Related: OS#5399 Change-Id: I4bd109009cbe16f313e19ab78bd5fbcb13b0f656 --- A .checkpatch.conf 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-e1d refs/changes/45/26945/1 diff --git a/.checkpatch.conf b/.checkpatch.conf new file mode 100644 index 0000000..20c0832 --- /dev/null +++ b/.checkpatch.conf @@ -0,0 +1,2 @@ +--ignore OPEN_BRACE +--ignore SPACING -- To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/26945 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1d Gerrit-Branch: master Gerrit-Change-Id: I4bd109009cbe16f313e19ab78bd5fbcb13b0f656 Gerrit-Change-Number: 26945 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

3 years, 6 months

1
0
0 0

Change in osmo-ci[master]: Revert "remove linter jobs for osmo-e1d and osmo-e1-firmware"

by osmith

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ci/+/26944 ) Change subject: Revert "remove linter jobs for osmo-e1d and osmo-e1-firmware" ...................................................................... Revert "remove linter jobs for osmo-e1d and osmo-e1-firmware" Enable the linter again, as we decided to use a repo-specific checkpatch config in these. This reverts commit 2c6aa82c9a8a81addd805fe211b048076e01f8c5. Related: OS#5399 Change-Id: Ib79a20658bfae64d9b718e53e6a9393708a4e694 --- M jobs/gerrit-lint.yml 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/44/26944/1 diff --git a/jobs/gerrit-lint.yml b/jobs/gerrit-lint.yml index b98a041..45db681 100644 --- a/jobs/gerrit-lint.yml +++ b/jobs/gerrit-lint.yml @@ -73,7 +73,9 @@ - simtrace2 - osmo-asf4-dfu - osmo-ccid-firmware + - osmo-e1d - osmo-cbc + - osmo-e1-recorder - gapk - osmo-uecups - osmo-el2tpd -- To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/26944 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ci Gerrit-Branch: master Gerrit-Change-Id: Ib79a20658bfae64d9b718e53e6a9393708a4e694 Gerrit-Change-Number: 26944 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

3 years, 6 months

1
0
0 0

Change in osmo-e1-hardware[master]: checkpatch.conf: new file

by osmith

osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-e1-hardware/+/26927 ) Change subject: checkpatch.conf: new file ...................................................................... checkpatch.conf: new file Disable spacing and open brace checks, as this repository follows a different code style. If needed, the checkpatch file can be extended with other checks: * get the check name from the linter output, e.g. SPACING in: firmware/ice40-riscv/icE1usb/fw_app.c:155: ERROR:SPACING: spaces required around that '=' (ctx:VxV) * add a new "--ignore NAME" line to the file Related: https://gerrit.osmocom.org/c/osmo-e1-hardware/+/26841 Related: OS#5399 Change-Id: I4d2cf2eb698b8f4e54f04e62e4dbbc37146daf88 --- A .checkpatch.conf 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-e1-hardware refs/changes/27/26927/1 diff --git a/.checkpatch.conf b/.checkpatch.conf new file mode 100644 index 0000000..20c0832 --- /dev/null +++ b/.checkpatch.conf @@ -0,0 +1,2 @@ +--ignore OPEN_BRACE +--ignore SPACING -- To view, visit https://gerrit.osmocom.org/c/osmo-e1-hardware/+/26927 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1-hardware Gerrit-Branch: master Gerrit-Change-Id: I4d2cf2eb698b8f4e54f04e62e4dbbc37146daf88 Gerrit-Change-Number: 26927 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: newchange

3 years, 7 months

3
3
0 0

2025

2024

2023

2022

gerrit-log