Fix more Warnings, and also a bug with checking the dirstructure for themes in rbutil

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@12862 a1c6a512-1295-4272-9138-f99709370657
This commit is contained in:
Dominik Wenger 2007-03-20 15:38:48 +00:00
parent 2671b6cda6
commit 22a673062f
4 changed files with 24 additions and 28 deletions

View file

@ -194,19 +194,17 @@ bool gigabeatf(int mode)
} }
path2 = path1; path2 = path1;
path2.Append(wxT(".ORIG")); path2.Append(wxT(".ORIG"));
if(wxFileExists(path2)) if(!wxFileExists(path2))
{ {
err = wxT("Its seems there is already a Bootloader install, if not, delete the *.IMG.ORIG file"); if(!wxRenameFile(path1,path2,false))
ERR_DIALOG(err, wxT("Bootloader add")); {
return false; err.Printf(wxT("[ERR] Coud not rename %s to %s"),path1.c_str(),path2.c_str());
ERR_DIALOG(err, wxT("Bootloader add"));
return false;
}
} }
if(!wxRenameFile(path1,path2,false))
{
err.Printf(wxT("[ERR] Coud not rename %s to %s"),path1.c_str(),path2.c_str());
ERR_DIALOG(err, wxT("Bootloader add"));
return false;
}
if(!wxCopyFile(dest,path1)) if(!wxCopyFile(dest,path1))
{ {
err.Printf(wxT("[ERR] Coud not copy %s to %s"),dest.c_str(),path2.c_str()); err.Printf(wxT("[ERR] Coud not copy %s to %s"),dest.c_str(),path2.c_str());
@ -300,7 +298,7 @@ bool h10(int mode)
if(!wxFileExists(path1)) //Firmware dosent exists on player if(!wxFileExists(path1)) //Firmware dosent exists on player
{ {
path1.Printf(wxT("%sSYSTEM" PATH_SEP "H10EMP.mi4")); //attempt other firmwarename path1.Printf(wxT("%sSYSTEM" PATH_SEP "H10EMP.mi4"),gv->curdestdir.c_str()); //attempt other firmwarename
if(!wxFileExists(path1)) //Firmware dosent exists on player if(!wxFileExists(path1)) //Firmware dosent exists on player
{ {
err.Printf(wxT("[ERR] File %s does not Exist"),path1.c_str()); err.Printf(wxT("[ERR] File %s does not Exist"),path1.c_str());
@ -308,18 +306,16 @@ bool h10(int mode)
return false; return false;
} }
} }
if(wxFileExists(path2)) //there is already a original firmware if(!wxFileExists(path2)) //there is already a original firmware
{ {
err.Printf(wxT("[ERR2] File %s does Exist"),path2.c_str()); if(!wxRenameFile(path1,path2,false)) //rename Firmware to Original
ERR_DIALOG(err, wxT("Bootloader add")); {
return false; err.Printf(wxT("[ERR] Coud not rename %s to %s"),path1.c_str(),path2.c_str());
} ERR_DIALOG(err, wxT("Bootloader add"));
if(!wxRenameFile(path1,path2,false)) //rename Firmware to Original return false;
{ }
err.Printf(wxT("[ERR] Coud not rename %s to %s"),path1.c_str(),path2.c_str());
ERR_DIALOG(err, wxT("Bootloader add"));
return false;
} }
if(!wxCopyFile(dest,path1)) // copy file if(!wxCopyFile(dest,path1)) // copy file
{ {
err.Printf(wxT("[ERR] Coud not copy %s to %s"),dest.c_str(),path1.c_str()); err.Printf(wxT("[ERR] Coud not copy %s to %s"),dest.c_str(),path1.c_str());
@ -336,7 +332,7 @@ bool h10(int mode)
path2.Printf(wxT("%sSYSTEM" PATH_SEP "Original.mi4"),gv->curdestdir.c_str()); path2.Printf(wxT("%sSYSTEM" PATH_SEP "Original.mi4"),gv->curdestdir.c_str());
if(!wxFileExists(path1)) //Firmware dosent exists on player if(!wxFileExists(path1)) //Firmware dosent exists on player
{ {
path1.Printf(wxT("%s" PATH_SEP "SYSTEM" PATH_SEP "H10EMP.mi4")); //attempt other firmwarename path1.Printf(wxT("%s" PATH_SEP "SYSTEM" PATH_SEP "H10EMP.mi4"),gv->curdestdir.c_str()); //attempt other firmwarename
if(!wxFileExists(path1)) //Firmware dosent exists on player if(!wxFileExists(path1)) //Firmware dosent exists on player
{ {
err.Printf(wxT("[ERR] File %s does not Exist"),path1.c_str()); err.Printf(wxT("[ERR] File %s does not Exist"),path1.c_str());

View file

@ -22,7 +22,7 @@
#define CREDITS_H_INCLUDED #define CREDITS_H_INCLUDED
#define RBUTIL_FULLNAME "The Rockbox Utility" #define RBUTIL_FULLNAME "The Rockbox Utility"
#define RBUTIL_VERSION "Version 0.3.0.1" #define RBUTIL_VERSION "Version 0.3.0.2"
static const wxString rbutil_developers[] = { static const wxString rbutil_developers[] = {
wxT("Christi Alice Scarborough"), wxT("Christi Alice Scarborough"),

View file

@ -10,8 +10,8 @@ BEGIN
BEGIN BEGIN
BLOCK "FFFF0000" BLOCK "FFFF0000"
BEGIN BEGIN
VALUE "FileVersion", "0.3.0.1\0" VALUE "FileVersion", "0.3.0.2\0"
VALUE "ProductVersion", "0.3.0.1\0" VALUE "ProductVersion", "0.3.0.2\0"
VALUE "CompanyName", "Rockbox Team\0" VALUE "CompanyName", "Rockbox Team\0"
VALUE "FileDescription", "Rockbox Utility\0" VALUE "FileDescription", "Rockbox Utility\0"
VALUE "InternalName", "rbutil\0" VALUE "InternalName", "rbutil\0"

View file

@ -105,7 +105,7 @@ bool checkZip(wxString zipname)
wxString name = entry->GetName(); wxString name = entry->GetName();
if(entry->IsDir()) if(entry->IsDir())
{ {
if(name.Contains(wxT(".rockbox\\"))) if(name.Contains(wxT(".rockbox")))
{ {
return true; return true;
} }
@ -126,6 +126,7 @@ int DownloadURL(wxString src, wxString dest)
wxPD_AUTO_HIDE | wxPD_SMOOTH | wxPD_ELAPSED_TIME | wxPD_AUTO_HIDE | wxPD_SMOOTH | wxPD_ELAPSED_TIME |
wxPD_REMAINING_TIME | wxPD_CAN_ABORT); wxPD_REMAINING_TIME | wxPD_CAN_ABORT);
progress->Update(0); progress->Update(0);
progress->SetSize(500,200);
input = true; input = true;
wxURL* in_http = new wxURL(src); wxURL* in_http = new wxURL(src);
@ -647,8 +648,7 @@ bool InstallRbutil(wxString dest)
if (! copied_exe) if (! copied_exe)
{ {
str.Printf(wxT("%s" PATH_SEP EXE_NAME), gv->AppDir.c_str()); str.Printf(wxT("%s" PATH_SEP EXE_NAME), gv->AppDir.c_str());
dstr.Printf(wxT("%s" PATH_SEP EXE_NAME), destdir.c_str(), dstr.Printf(wxT("%s" PATH_SEP EXE_NAME), destdir.c_str());
filestocopy[i].AfterLast(PATH_SEP_CHR).c_str());
if (! wxCopyFile(str, dstr) ) if (! wxCopyFile(str, dstr) )
{ {
buf.Printf(wxT("Can't copy program binary %s -> %s"), buf.Printf(wxT("Can't copy program binary %s -> %s"),