| Summary: | submit should not use authenticated ssh to do the svn checkout | ||
|---|---|---|---|
| Product: | Infrastructure | Reporter: | Michael Scherer <misc> |
| Component: | BuildSystem | Assignee: | Sysadmin Team <sysadmin-bugs> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | Normal | CC: | marja11, sysadmin-bugs |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Hardware: | i586 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Source RPM: | CVE: | ||
| Status comment: | |||
| Bug Depends on: | |||
| Bug Blocks: | 858 | ||
| Attachments: | Script in perl to replace old bash script | ||
|
Description
Michael Scherer
2011-04-21 12:47:37 CEST
Michael Scherer
2011-04-21 12:48:42 CEST
Blocks:
(none) =>
858 On simple way would be to do a replacement in submit_package , just before the exec : $* is "-t cauldron --define sid=7ef09d9e-fd3b-49e7-9d2a-06354a0f4f42 -r 89293 svn+ssh://svn.mageia.org/svn/packages/cauldron/null" Maybe we should rewrite it to use something else than bash ( cause I see not elegant way to do 's|^svn+ssh://|svn://|' ) Created attachment 268 [details]
Script in perl to replace old bash script
Here is a proposal for a the replacement.
But a better solution would be to not have mgarepo to give the url, but maybe to give the information needed to find it, or maybe to have mgarepo do the filtering job itself.
(In reply to comment #2) > Created attachment 268 [details] > Script in perl to replace old bash script > > Here is a proposal for a the replacement. > > But a better solution would be to not have mgarepo to give the url, but maybe > to give the information needed to find it, or maybe to have mgarepo do the > filtering job itself. Yes, I was thinking about doing that. Not give a URL but only the name of the package. And add an option to specify a branch (as I think the reason for giving an URL was to be able to use branches). What is described in this mail : http://www.mageia.org/pipermail/mageia-dev/20110129/002372.html But I didn't take time to do it yet. But we can use your script as a temporary solution. CC:
(none) =>
boklm Ok, I deployed it today, we still need a better fix. (In reply to comment #4) > Ok, I deployed it today, we still need a better fix. Any news? CC:
(none) =>
marja11 Pinging. because nothing happened to this report since more than 3 months ago, and it still has the status NEW or REOPENED. Please set status to ASSIGNED. If for work flow reasons you can't do that, then please put OK on the whiteboard instead.
Manuel Hiebel
2012-01-28 15:23:19 CET
Assignee:
mageia-sysadm =>
sysadmin-bugs Submit script replace svn+ssh:// with svn:// in the URL. Status:
NEW =>
RESOLVED
Nicolas Vigier
2014-05-08 18:04:58 CEST
CC:
boklm =>
(none) |