Page 1 of 1

Autoupdate.pl error | r6927

Posted: 30 Oct 2009, 01:50
by 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 :D

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

Thanks 8-)

Re: Autoupdate.pl error | r6927

Posted: 30 Oct 2009, 06:26
by kali
Could it be your svn version does not have that particular switch enabled?

Re: Autoupdate.pl error | r6927

Posted: 30 Oct 2009, 14:43
by zarkon
kali wrote: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

Re: Autoupdate.pl error | r6927

Posted: 30 Oct 2009, 17:39
by kLabMouse
Or I'm Blind. OR "update\-\-force" has no space between?

Re: Autoupdate.pl error | r6927

Posted: 30 Oct 2009, 17:50
by kLabMouse
Fixed in r6929.
Try it out.

Re: Autoupdate.pl error | r6927

Posted: 30 Oct 2009, 19:10
by 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?

Re: Autoupdate.pl error | r6927

Posted: 31 Oct 2009, 03:02
by 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

Re: Autoupdate.pl error | r6927

Posted: 17 Nov 2009, 10:20
by kLabMouse
Hmm...
gimme output of "svn -v"
to know the version of SVN tool used by you.

Re: Autoupdate.pl error | r6927

Posted: 22 Dec 2009, 17:30
by EternalHarvest
"--accept theirs-conflict" fixed.

Re: Autoupdate.pl error | r6927

Posted: 22 Dec 2009, 18:14
by kLabMouse
Closed.