Bug 16362 - mgarepo could suggest to svn ci in appropriate cases
Summary: mgarepo could suggest to svn ci in appropriate cases
Status: NEW
Alias: None
Product: Infrastructure
Classification: Unclassified
Component: Others (show other bugs)
Version: unspecified
Hardware: All Linux
Priority: Normal enhancement
Target Milestone: ---
Assignee: Sysadmin Team
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-12 23:27 CEST by Bruno Cornec
Modified: 2015-12-22 13:42 CET (History)
3 users (show)

See Also:
Source RPM: mgarepo-1.10.6-5.mga5
CVE:
Status comment:


Attachments

Description Bruno Cornec 2015-07-12 23:27:47 CEST
Description of problem:

When doing a mgarepo submit and that there is a svn diff/status in your environment, a warning could be emitted to allow the user to check in before submitting as this is generally what you want to do.



Reproducible: 

Steps to Reproduce:
Comment 1 Florian Hubold 2015-12-19 14:31:08 CET
So each time a svn diff exists, mgarepo should issue a warning that a diff exists when you run mgarepo ci?

You are required to check what you commit with svn diff beforehand, so this pretty much makes no real sense if you want to reverse this. Otherwise we end up with more "do you really want to do this" and "are you really sure" dialogs like in windows vista or later with raised UAC ...

Do you have some actual example for this - and how do you define appropriate cases where such a warning should be issued by mgarepo?

CC: (none) => doktor5000

Florian Hubold 2015-12-19 14:32:07 CET

See Also: (none) => https://bugs.mageia.org/show_bug.cgi?id=3435

Comment 2 Bruno Cornec 2015-12-22 11:48:02 CET
The major use case I had for this was when you update a package following an upstream update (you move say from 1.0 to 1.1), you update the spec file accordingly, do mgarepo sync -d to get the content in sync, you build with bm and then you do mgarepo submit, forgetting to do a svn ci before. I'd like mgarepo to be able to detect it and propose to me to do the svn/mgarepo ci if I validate it.
Comment 3 Angelo Naselli 2015-12-22 13:42:22 CET
@Florian iiuc the case is before submitting. So if there are differences magarepo should prevent the submission. The svn diff/status case should be easy to test,
but I'm not sure that is also easy if a patch or a source pacakge has not been
added though... But managing at least the easier tasks should be done anyway...

CC: (none) => anaselli


Note You need to log in before you can comment on or make changes to this bug.