Bug 25417

Summary: [FR] some feedback on no-ops
Product: Sisyphus Reporter: Michael Shigorin <mike>
Component: apt-repoAssignee: Andrey Cherepanov <cas>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: enhancement    
Priority: P3 CC: cas, dd1email
Version: unstable   
Hardware: all   
OS: Linux   

Description Michael Shigorin 2011-04-11 17:37:14 MSK
Хорошо бы или делать молча, или с данными аргументами делать ничего не пришлось -- то как-то об этом сообщить.  Запись пробной сессии as is:

root@pad ~ # apt-repo 
rpm file:/var/ftp/pub/Linux/ALT/Sisyphus i586 classic
rpm file:/var/ftp/pub/Linux/ALT/Sisyphus noarch classic
root@pad ~ # apt-repo --help
Usage:
     apt-repo> [list] [-a]
     apt-repo> add [APT|URL|branch|task] [name or component]
     apt-repo> rm [APT|URL|branch|task] [name or component]
     apt-repo> update

root@pad ~ # apt-repo rm rpm file:/var/ftp/pub/Linux/ALT/Sisyphus noarch classic
Nothing to remove: bad repository format. See -h for details.
root@pad ~ # apt-repo rm file:/var/ftp/pub/Linux/ALT/Sisyphus noarch classic    
root@pad ~ # apt-repo 
rpm file:/var/ftp/pub/Linux/ALT/Sisyphus i586 classic
rpm file:/var/ftp/pub/Linux/ALT/Sisyphus noarch classic
root@pad ~ # apt-repo rm file:/var/ftp/pub/Linux/ALT/Sisyphus noarch classic
root@pad ~ # apt-repo                                                       
rpm file:/var/ftp/pub/Linux/ALT/Sisyphus i586 classic
rpm file:/var/ftp/pub/Linux/ALT/Sisyphus noarch classic
root@pad ~ # apt-repo rm file:/var/ftp/pub/Linux/ALT/Sisyphus               
root@pad ~ # apt-repo                                        
root@pad ~ # apt-repo add file:/var/ftp/pub/Linux/ALT/Sisyphus noarch classic
root@pad ~ # apt-repo                                                        
rpm file:/var/ftp/pub/Linux/ALT/Sisyphus i586 noarch
rpm file:/var/ftp/pub/Linux/ALT/Sisyphus noarch noarch
root@pad ~ # 

Команда apt-repo rm file:/var/ftp/pub/Linux/ALT/Sisyphus noarch classic никаких изменений не внесла (что само по себе не баг), предупреждений не вынесла (а вот тут бы поправить).  Также и apt-repo rm branch Sisyphus лучше бы предупредило -- "nothing to do" или что-нить вроде того.

Про "noarch noarch" сейчас отдельно повешу.
Comment 1 Andrey Cherepanov 2011-04-11 18:46:11 MSK
Взял на вооружение.
Comment 2 Repository Robot 2011-05-09 09:48:23 MSK
apt-repo-1.0.2-alt1 -> sisyphus:

* Mon May 09 2011 Andrey Cherepanov <cas@altlinux> 1.0.2-alt1
- Make error messages more informational (closes: #25417)
- Inform about missed task number
- Show all available branch names
- Complete documentation
- Support sources.list(5) tokens in command line
- Pass all arguments as parts of source line (closes: #25418)
- Support quick forms of source: known branch name or number for task
- Fix URL for Sisyphus. Support absolute pathname for hasher repo.