Autoupdate.pl error | r6927

[zarkon]

error when running autoupdate.pl

Code: Select all

$ perl autoupdate.pl 
-===================== OpenKore Auto Update tool =====================-
Checking OpenKore core files for updates...
  Fetching updates...
Unable to do svn update\-\-force \-\-accept\ theirs\-conflict .
svn: invalid option: --accept theirs-conflict
Type 'svn help' for usage.
 at /home/mohano/games/openkore-2.0.7/src/deps/SVN/Updater.pm line 18
	SVN::Updater::_svn_command('SVN::Updater=HASH(0xa066188)', 'update', '--force', '--accept theirs-conflict') called at /home/mohano/games/openkore-2.0.7/src/deps/SVN/Updater.pm line 90
	SVN::Updater::update('SVN::Updater=HASH(0xa066188)', '--force', '--accept theirs-conflict') called at autoupdate.pl line 41
	autoupdate::upgrade(., 'OpenKore core files') called at autoupdate.pl line 47

Note

Code: Select all

svn update --force --accept thier-conflict 

is a valid working command so we can rules out syntax errors/typos

script run on
Debian 5.0.3 Lenny
Perl v5.10.0 built for i486-linux-gnu-thread-multi

Thanks

[kali]

Could it be your svn version does not have that particular switch enabled?

[zarkon]

Could it be your svn version does not have that particular switch enabled?

nope my svn have that switch tested by running manually it does work even i removed --accept leave --force it still show the same error

[kLabMouse]

Or I'm Blind. OR "update\-\-force" has no space between?

[kLabMouse]

Fixed in r6929.
Try it out.

[EternalHarvest]

Code: Select all

# Updates current working directory from the latest repository contents.
sub update {
	my ($self, @params) = @_;
	$self->_svn_command('cleanup'); # Cleaup Repos, if something go wrong
	return $self->_svn_command('update ', @params);
}

# Diffs the file against the repository.
sub diff {
	return join('', shift()->_svn_command('diff', @_));
}

# Checks-out working copy from the REPOSITORY into directory given by 'path' option.
sub checkout {
	my ($self, $repository) = @_;
	mkdir($self->path) or confess "Unable to create " . $self->path;
	return $self->_svn_command('checkout', $repository, '.');
}

How about spaces in diff and checkout? Do they work?

[zarkon]

Code: Select all

-===================== OpenKore Auto Update tool =====================-
Checking OpenKore core files for updates...
  Fetching updates...
Unable to do svn update  \-\-force \-\-accept\ theirs\-conflict .
svn: invalid option: --accept theirs-conflict
Type 'svn help' for usage.
 at /home/mohano/games/openkore-2.0.7/src/deps/SVN/Updater.pm line 18
	SVN::Updater::_svn_command('SVN::Updater=HASH(0x9d1da50)', 'update ', '--force', '--accept theirs-conflict') called at /home/mohano/games/openkore-2.0.7/src/deps/SVN/Updater.pm line 90
	SVN::Updater::update('SVN::Updater=HASH(0x9d1da50)', '--force', '--accept theirs-conflict') called at autoupdate.pl line 41
	autoupdate::upgrade(., 'OpenKore core files') called at autoupdate.pl line 47

my svn have the option:

Code: Select all

  --accept ARG             : specify automatic conflict resolution action
                            ('postpone', 'base', 'mine-conflict',
                             'theirs-conflict', 'mine-full', 'theirs-full',
                             'edit', 'launch')

seem it does not regconize the option, i manage to fix it by changing how autoupdate send the command instead using:

Code: Select all

$sa->update("--force", "--accept theirs-conflict");

which the script does not regconize the --accept option i change it to:

Code: Select all

$sa->update("--force", "--accept=theirs-conflict");

it work now, seem autoupdate.pl or maybe Updater.pm can parse a SVN option that contain space in it, i tested with other svn option that have space in it like --depth infinity it will end up showing option is in invalid by the autoupdater

Cheers

[kLabMouse]

Hmm...
gimme output of "svn -v"
to know the version of SVN tool used by you.

[EternalHarvest]

"--accept theirs-conflict" fixed.

[kLabMouse]

Closed.