0001453: wget should use "--continue"!

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0001453

USAF

[All Projects] General

public

2015-12-09 02:23

2016-11-22 10:52

Reporter

rogerx

Assigned To

Soul_Assassin

Priority

none

Severity

minor

Reproducibility

always

Status

closed

Resolution

won't fix

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0001453: wget should use "--continue"!

Description

FILE: update_rhsusf.bat

-- wget -m -nH --cut-dirs=2 --retry-connrefused --timeout=30 ftp://ftp.rhsmods.org/beta/rhsusaf/ [^]

++ wget --continue --cut-dirs=2 --mirror --no-host-directories --retry-connrefused --timeout=30 ftp://ftp.rhsmods.org/beta/rhsusaf/ [^]

1) Added "--continue" parameter. (For obvious reasons, stopping a download should not have to revert to downloading for the first byte! Reduces server loads.)

2) I also switched "-nH" short option for the long option "--no-host-directories". The same with "-m", the long option being "--mirror". A underlying rule for short/long options, use the short options when typing via command line for less typing, and the long options for use within script files for easier reading.

3) Alphabetized wget's used parameters.

Implementation of "--continue" should likely be the priority, versus the other changes which are basically cosmetic, but dictate good coding style. ;-)

Tags

No tags attached.

Is it a wish/request?

RHS version

Arma 3 version

1.54

Did you used any other mod when the error occurred?

Which mods?

Attached Files

Relationships

Notes
(0002987) Soul_Assassin (administrator) 2015-12-09 11:15	Thanks for the input. Some feedback: 1. It was actually initially there, however for some reason it did not work with the pbos, and they became corrupted, and thus this was removed. 2. While I agree about style changes it does not warrant right now for immediate update. With that being said I leave this task on "Acknowledged" as I think these comments are valid. I had in mind already for a while a more advanced updater (same background wget tech but nicer interface/feedback). When I get around to it I will keep these things in mind.

(0005551) Soul_Assassin (administrator) 2016-11-22 10:52	In light of full move to SW, this is irrelevant.

Issue History
Date Modified	Username	Field	Change
2015-12-09 02:23	rogerx	New Issue
2015-12-09 08:19	RedPhoenix	Assigned To	=> Soul_Assassin
2015-12-09 11:15	Soul_Assassin	Note Added: 0002987
2015-12-09 11:15	Soul_Assassin	Status	new => acknowledged
2016-11-22 10:52	Soul_Assassin	Note Added: 0005551
2016-11-22 10:52	Soul_Assassin	Status	acknowledged => closed
2016-11-22 10:52	Soul_Assassin	Resolution	open => won't fix