Mailinglist Archive: opensuse-buildservice (348 mails)
| < Previous | Next > |
Re: [opensuse-buildservice] Re: [PATCH] - allow diff against a nonexistent (source/target) project/package
- From: Michael Schroeder <mls@xxxxxxx>
- Date: Mon, 12 Apr 2010 12:43:27 +0200
- Message-id: <20100412104327.GA2843@xxxxxxx>
On Mon, Apr 12, 2010 at 11:39:13AM +0200, Marcus Hüwe wrote:
Why does remote_getrev() behave different than standard getrev() in
the first place? Shouldn't 'missingok' be the default?
M.
--
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-buildservice+help@xxxxxxxxxxxx
On Mon, Apr 12, 2010 at 10:38:49AM +0200, Michael Schroeder wrote:
On Sat, Apr 10, 2010 at 03:28:07PM +0200, OBS build-service wrote:I want to support a diff against a non-existent project/package. For instance
From: Marcus Huewe <suse-tux@xxxxxx>
[...]
- my ($projid, $packid, $rev, $linked) = @_;
+ my ($projid, $packid, $rev, $linked, $missingok) = @_;
my $proj = remoteprojid($projid);
if (!$proj) {
+ return {'project' => $projid, 'package' => $packid, 'srcmd5' =>
'empty'} if $missingok;
[...]
What exactly are you trying to solv with that flag?
if
you create a new submitrequest which would create a new package in the target
project you won't be able to view the diff because the package doesn't exist
in
the target project. The missingok flag is used to tell remote_getrev() not to
die if the project and/or package doesn't exist.
Why does remote_getrev() behave different than standard getrev() in
the first place? Shouldn't 'missingok' be the default?
M.
--
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-buildservice+help@xxxxxxxxxxxx
| < Previous | Next > |