Commit a9b12cae authored by Felix Paul Kühne's avatar Felix Paul Kühne

* made the look-for-update process faster and more error tolerant

    - we can't get release notes if vlc-version == current-release, so don't wait for them
    - wait only 15 sec for the r-notes, any longer can't be good
    - show the user that her version is up-to-date
parent bc38c3ea
...@@ -140,12 +140,11 @@ static VLCUpdate *_o_sharedInstance = nil; ...@@ -140,12 +140,11 @@ static VLCUpdate *_o_sharedInstance = nil;
update_Check( p_u, VLC_FALSE ); update_Check( p_u, VLC_FALSE );
update_iterator_t *p_uit = update_iterator_New( p_u ); update_iterator_t *p_uit = update_iterator_New( p_u );
BOOL releaseChecked = NO; BOOL releaseChecked = NO;
BOOL gettingReleaseNote = NO;
int x = 0; int x = 0;
NSString * pathToReleaseNote; NSString * pathToReleaseNote;
pathToReleaseNote = [NSString stringWithFormat: \ pathToReleaseNote = [NSString stringWithFormat: \
@"/tmp/vlc_releasenote_%d.tmp", mdate()]; /*[[NSCalendarDate calendarDate] \ @"/tmp/vlc_releasenote_%d.tmp", mdate()];
descriptionWithCalendarFormat: @"%m-%d-%y--%I.%M.%S.%F"]];*/
NSLog( pathToReleaseNote );
if( p_uit ) if( p_uit )
{ {
...@@ -155,6 +154,7 @@ static VLCUpdate *_o_sharedInstance = nil; ...@@ -155,6 +154,7 @@ static VLCUpdate *_o_sharedInstance = nil;
while( update_iterator_Action( p_uit, UPDATE_FILE) != UPDATE_FAIL ) while( update_iterator_Action( p_uit, UPDATE_FILE) != UPDATE_FAIL )
{ {
msg_Dbg( p_intf, "parsing available updates, run %i", x );
/* if the announced item is of the type "binary", keep it and display /* if the announced item is of the type "binary", keep it and display
* its details to the user. Do similar stuff on "info". Do both * its details to the user. Do similar stuff on "info". Do both
* only if the file is announced as stable */ * only if the file is announced as stable */
...@@ -162,12 +162,15 @@ static VLCUpdate *_o_sharedInstance = nil; ...@@ -162,12 +162,15 @@ static VLCUpdate *_o_sharedInstance = nil;
{ {
if( p_uit->file.i_type == UPDATE_FILE_TYPE_INFO ) if( p_uit->file.i_type == UPDATE_FILE_TYPE_INFO )
{ {
msg_Dbg( p_intf, "release note found, desc = %s",
p_uit->file.psz_description );
[o_fld_releaseNote setString: \ [o_fld_releaseNote setString: \
[NSString stringWithUTF8String: \ [NSString stringWithUTF8String: \
(p_uit->file.psz_description)]]; (p_uit->file.psz_description)]];
/* download our release note /* download our release note
* We will read the temp file after this loop */ * We will read the temp file after this loop */
update_download( p_uit, (char *)[pathToReleaseNote UTF8String] ); update_download( p_uit, (char *)[pathToReleaseNote UTF8String] );
gettingReleaseNote = YES;
} }
else if( p_uit->file.i_type == UPDATE_FILE_TYPE_BINARY ) else if( p_uit->file.i_type == UPDATE_FILE_TYPE_BINARY )
{ {
...@@ -217,18 +220,20 @@ static VLCUpdate *_o_sharedInstance = nil; ...@@ -217,18 +220,20 @@ static VLCUpdate *_o_sharedInstance = nil;
update_iterator_Delete( p_uit ); update_iterator_Delete( p_uit );
/* wait for our download, since it is done by another thread /* wait for our release notes if necessary, since the download is done
* this does usually take 300000 to 500000 ms */ * by another thread -- this does usually take 300000 to 500000 ms */
if( gettingReleaseNote )
{
int i = 0; int i = 0;
while( [[NSFileManager defaultManager] fileExistsAtPath: pathToReleaseNote] == NO ) while( [[NSFileManager defaultManager] fileExistsAtPath: pathToReleaseNote] == NO )
{ {
msleep( 100000 ); msleep( 100000 );
i += 1; i += 1;
if( i == 600 ) if( i == 150 )
{ {
/* if this takes more than 1 min, exit */ /* if this takes more than 15 sec, exit */
msg_Warn( p_intf, "download took more than a minute, exiting" ); msg_Warn( p_intf, "download took more than 15 sec, exiting" );
return; break;
} }
} }
msg_Dbg( p_intf, "waited %i ms for the release notes", (i * 100000) ); msg_Dbg( p_intf, "waited %i ms for the release notes", (i * 100000) );
...@@ -248,6 +253,16 @@ static VLCUpdate *_o_sharedInstance = nil; ...@@ -248,6 +253,16 @@ static VLCUpdate *_o_sharedInstance = nil;
myBOOL = [[NSFileManager defaultManager] removeFileAtPath: \ myBOOL = [[NSFileManager defaultManager] removeFileAtPath: \
pathToReleaseNote handler: nil]; pathToReleaseNote handler: nil];
} }
else
{
/* don't confuse the user, but make her happy */
[o_fld_status setStringValue: _NS("This version of VLC " \
"is latest available.")];
[o_btn_DownloadNow setEnabled: NO];
msg_Dbg( p_intf, "current version is up-to-date" );
msg_Warn( p_intf, "retrieving current release notes failed!" );
}
}
} }
- (void)performDownload:(NSString *)path - (void)performDownload:(NSString *)path
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment