Attention is currently required from: neels, laforge.
osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/29110 )

Change subject: obs: add README
......................................................................


Patch Set 3:

(4 comments)

Patchset:

PS3:
very nice, thanks for adding this!


File scripts/obs/README:

https://gerrit.osmocom.org/c/osmo-ci/+/29110/comment/35bf3115_76c8b762
PS3, Line 6:  apt-get install dh-python meson osc rebar3
There are more dependencies, the debian package names are listed in 
data/Dockerfile. Maybe just reference that? But I don't have a strong opinion 
on this, if you think it's useful then let's duplicate it here.


https://gerrit.osmocom.org/c/osmo-ci/+/29110/comment/a23675a8_1a32f3bc
PS3, Line 26: Easiest is to let osc create one and then edit it:
why not provide an example ~/.oscrc instead of these instructions?

  [general]
  # apiurl = https://api.opensuse.org
  apiurl = https://obs.osmocom.org

  # [https://api.opensuse.org]
  # user = user-for-opensuse-obs
  # pass =

  [https://obs.osmocom.org]
  user = user-for-osmocom-obs
  pass =


https://gerrit.osmocom.org/c/osmo-ci/+/29110/comment/ebb74a06_d2067198
PS3, Line 35: Replace "api.opensuse.org" with "obs.osmocom.org" in two places:
> Isn't the "-A https://lobs.osmocom. […]
ack, if using "osc -A https://obs.osmocom.org list ..." it will probably 
generate an oscrc against that server and the sed isn't needed



--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/29110
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I4d1303e0c04e827200b48a9fe4aea3680c9c9f84
Gerrit-Change-Number: 29110
Gerrit-PatchSet: 3
Gerrit-Owner: neels <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: neels <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Comment-Date: Wed, 17 Aug 2022 09:15:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Gerrit-MessageType: comment

Reply via email to