Bug 26239 - univention-repository-create sollte --iso unterstützen
univention-repository-create sollte --iso unterstützen
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Update - univention-updater
UCS 3.0
All Linux
: P5 enhancement (vote)
: UCS 3.1
Assigned To: Philipp Hahn
Florian Best
: interim-3
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-02-24 08:35 CET by Philipp Hahn
Modified: 2012-12-12 21:07 CET (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Further conceptual development
Max CVSS v3 score:
hahn: Patch_Available+


Attachments
univention-repository-create --iso (1.55 KB, patch)
2012-02-24 08:35 CET, Philipp Hahn
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2012-02-24 08:35:12 CET
Created attachment 4218 [details]
univention-repository-create --iso

univention-upgrade und univention-repository-update unterstützen jeweils die
Verwendung eines ISO-Images. Aus Konsistenzgründen wäre es wünschenswert, wenn
univention-repository-create auch von einem ISO-Image aus funktionieren würde.

(System wurde von USB-Stick bzw. PXE installier, so daß kein DVD-Laufwerk
vorhanden ist, wohl aber inzwischen die ISOs zwecks Virtualisierung)
Comment 1 Philipp Hahn univentionstaff 2012-08-07 21:16:38 CEST
Patch wurde eingespielt.

svn34692, univention-updater_8.0.2-1.1066.201208072113

ChangeLog: svn
\item \ucsCommand{univention-repository-update} now supports the \emph{-{}-iso} option to update a local repository from a downloaded \emph{.iso} file (\ucsBug{26239}).
Comment 2 Stefan Gohmann univentionstaff 2012-09-12 11:29:12 CEST
Da scheint etwas beim commit schief gegangen zu sein:

root@master131:~# univention-repository-create 
  File "/usr/sbin/univention-repository-create", line 233
    ( options, arguments ) = parser.parse_args()
    ^
IndentationError: unexpected indent


@@ -223,11 +227,13 @@
        parser.add_option( '-m', '--mount-point', action = 'store',
                                           dest = 'mount_point', default = '/cdrom',
                                           help = 'devices mount point for CD-ROM drive' )
++      parser.add_option('-i', '--iso', action='store', dest='iso',
++                        default=None, help='define filename of an ISO image')
 
        ( options, arguments ) = parser.parse_args()
 
        if options.silent:
-               sys.stdout = open( '/dev/null', 'w' )
++              sys.stdout = open(os.path.devnull, 'w')
 
        try:
                lock = univention.updater.tools.updater_lock_acquire()
Comment 3 Philipp Hahn univentionstaff 2012-09-12 12:44:10 CEST
(In reply to comment #2)
> Da scheint etwas beim commit schief gegangen zu sein:
> -               sys.stdout = open( '/dev/null', 'w' )
> ++              sys.stdout = open(os.path.devnull, 'w')

Ja, die + sind zu viel und wurden entfernt.

svn35548, univention-updater_8.0.11-1.1074.201209121241
ChangeLog: ±0
Comment 4 Sönke Schwardt-Krummrich univentionstaff 2012-09-12 12:55:15 CEST
root@master:~# univention-repository-create 
  File "/usr/sbin/univention-repository-create", line 233
    ( options, arguments ) = parser.parse_args()
    ^
IndentationError: unexpected indent
root@master:~# 

Im betreffenden Codeabschnitt fangen drei Python-Zeilen mit "+" an.
Comment 5 Sönke Schwardt-Krummrich univentionstaff 2012-09-12 12:56:22 CEST
(In reply to comment #4)
> Im betreffenden Codeabschnitt fangen drei Python-Zeilen mit "+" an.

War zu langsam.
Comment 6 Florian Best univentionstaff 2012-11-20 11:08:40 CET
OK, das repository wird korrekt aus dem .iso image erstellt.

Changelog: OK, typo wurde beseitigt (\emph{-{}-iso} → \emph{--iso}).
Comment 7 Stefan Gohmann univentionstaff 2012-12-12 21:07:45 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

If this error occurs again, please use "Clone This Bug".