[S] Change in ...osmo-epdg[master]: jenkins.sh: use available Makefile, clean before building

2024-03-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email )

Change subject: jenkins.sh: use available Makefile, clean before building
..

jenkins.sh: use available Makefile, clean before building

The project needs to go through clean procedure to clean up the
generated diameter files, so that they are generated from newer dia
files again.

Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
---
M contrib/jenkins.sh
1 file changed, 16 insertions(+), 3 deletions(-)

Approvals:
  laforge: Looks good to me, but someone else must approve
  Jenkins Builder: Verified
  pespin: Looks good to me, approved




diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 87eae34..a900a5c 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -1,5 +1,5 @@
 #!/bin/sh -ex

-rebar3 compile
-rebar3 escriptize
-rebar3 eunit
+make clean || true
+make
+make check

--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
Gerrit-Change-Number: 36264
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


[S] Change in ...osmo-epdg[master]: jenkins.sh: use available Makefile, clean before building

2024-03-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email )

Change subject: jenkins.sh: use available Makefile, clean before building
..


Patch Set 2: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
Gerrit-Change-Number: 36264
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Wed, 13 Mar 2024 23:08:18 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


[S] Change in ...osmo-epdg[master]: jenkins.sh: use available Makefile, clean before building

2024-03-13 Thread laforge
Attention is currently required from: pespin.

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email )

Change subject: jenkins.sh: use available Makefile, clean before building
..


Patch Set 2: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
Gerrit-Change-Number: 36264
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Attention: pespin 
Gerrit-Comment-Date: Wed, 13 Mar 2024 21:33:02 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


[S] Change in ...osmo-epdg[master]: jenkins.sh: use available Makefile, clean before building

2024-03-13 Thread pespin
Attention is currently required from: pespin.

Hello Jenkins Builder,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email

to look at the new patch set (#2).

The following approvals got outdated and were removed:
Verified-1 by Jenkins Builder


Change subject: jenkins.sh: use available Makefile, clean before building
..

jenkins.sh: use available Makefile, clean before building

The project needs to go through clean procedure to clean up the
generated diameter files, so that they are generated from newer dia
files again.

Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
---
M contrib/jenkins.sh
1 file changed, 16 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-epdg 
refs/changes/64/36264/2
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
Gerrit-Change-Number: 36264
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin 
Gerrit-MessageType: newpatchset


[S] Change in ...osmo-epdg[master]: jenkins.sh: use available Makefile, clean before building

2024-03-13 Thread Jenkins Builder
Jenkins Builder has posted comments on this change. ( 
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email )

Change subject: jenkins.sh: use available Makefile, clean before building
..


Patch Set 1:

(1 comment)

File contrib/jenkins.sh:

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-15078):
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264/comment/03dacf3d_88306a44
PS1, Line 5: make check
adding a line without newline at end of file



--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
Gerrit-Change-Number: 36264
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Wed, 13 Mar 2024 19:10:14 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


[S] Change in ...osmo-epdg[master]: jenkins.sh: use available Makefile, clean before building

2024-03-13 Thread pespin
pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email )


Change subject: jenkins.sh: use available Makefile, clean before building
..

jenkins.sh: use available Makefile, clean before building

The project needs to go through clean procedure to clean up the
generated diameter files, so that they are generated from newer dia
files again.

Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
---
M contrib/jenkins.sh
1 file changed, 16 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-epdg 
refs/changes/64/36264/1

diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 87eae34..c199021 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -1,5 +1,5 @@
 #!/bin/sh -ex

-rebar3 compile
-rebar3 escriptize
-rebar3 eunit
+make clean || true
+make
+make check
\ No newline at end of file

--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36264?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: Id3f2d71c184cf387fa178d5138d401686a8d0a63
Gerrit-Change-Number: 36264
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-MessageType: newchange