Maintenance requests may be done from a working copy of a package
to just update a single code stream at once.
In this situation, this working copy may have been used to implement
this change. Thus, like with submit requestes, check if the working
copy has local changes that have not been checked in and warn the
user about it. This should help to prevent resource-intensive accidents.
Signed-off-by: Egbert Eich <eich@suse.com>
Fix these by adding a default empty assignment:
osc/commandline.py:3382:62: E0606: Possibly using variable 'filter_pattern' before assignment (possibly-used-before-assignment)
osc/commandline.py:6692:18: E0606: Possibly using variable 'msg' before assignment (possibly-used-before-assignment)
osc/core.py:2051:23: E0606: Possibly using variable 'file_changed' before assignment (possibly-used-before-assignment)
These could probably be fixed by making the code easier to follow. By
silencing them for now and making the CI pass, it is easier to not
regress.
osc/commandline.py:3269:55: E0601: Using variable 'incident' before assignment (used-before-assignment)
osc/commandline.py:3276:55: E0601: Using variable 'priority' before assignment (used-before-assignment)
This is pylint not understanding that the try is for catching this case
and only importing when available:
osc/conf.py:77:4: E0401: Unable to import 'keyring' (import-error)
Doing the assignment conditionally doesn't make sense here, so move it
outside the condition.
osc/commandline.py:8311:15: E0606: Possibly using variable 'p' before assignment (possibly-used-before-assignment)
This always throws an exception, so anntotating it NoReturn allows
pylint to notice some exhaustive choice, removing this false errors:
osc/commandline.py:6581:78: E0606: Possibly using variable 'arch' before assignment (possibly-used-before-assignment)
The exception above would have already triggered otherwise. Thus
removing this pylint error:
osc/commandline.py:4371:38: E0601: Using variable 'package' before assignment (used-before-assignment)
Allow passing `--multibuild-package` or `-M` to the rpmlint command to
specify flavors for a multibuild package. This makes it consistent with
other commands, for example `buildlog`, where this option is similarly
used.
Commit 083fdf3b introduced a feature for the 'submitrequest' command
where the submission description is pre-populated with one of a superseded
request.
Here, the same is introduced for the `maintenancerequest` command.
The behavior of `osc mr -s 1234` is identical to `osc sr -s 1234` - see Issue 1575.
Signed-off-by: Egbert Eich <eich@suse.com>
This allows to skip the check of obs-scm-bridge and checks out what
is stored in OBS src server.
Note: This is not necessarly the same representation as in git
(.obscpio instead of directories or sub modules).
Also no modification can be committed.
So this is only intended for people who understand the current
internal implementation of the source storage. There is no
guarantee that this won't change.
Checkout of entire project git is not working with this switch, as
the packagelist of osc is empty. It may also make no sense to do
this as it will not perform for large projects.
Unlike other recipe files, OBS detects this by matching the prefix rather
than the suffix. As this would cause conflicts with e.g. mkosi.spec the
build type detection had to be changed a bit.
This used to be printed but was dropped with a refactoring.
Of course, there is the --just-print-buildroot option, but
there is no harm printing this information anyway.
Signed-off-by: Egbert Eich <eich@suse.com>
Add --no-timestamps parameter which already exists in build.
Help text copied from build.
Include parameters for the same parameter in buildlog for easier
use.
There are two ways, either copy the scmsync definition or drop it
when creating a new package meta.
If we keep it, we must not ask OBS to copy sources, it is syncing it
anyway.
We could add another option to skip the scmsync tag copy, but we keep
it as default, because we don't want to give a different view of the
the sources to the user. The client side copy is doing this.
