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