Merge lp:~manishsinha/software-properties/add-update-after-repo-add into lp:software-properties
Proposed by
Manish Sinha (मनीष सिन्हा)
Status: | Needs review | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~manishsinha/software-properties/add-update-after-repo-add | ||||
Merge into: | lp:software-properties | ||||
Diff against target: |
53 lines (+25/-0) 1 file modified
add-apt-repository (+25/-0) |
||||
To merge this branch: | bzr merge lp:~manishsinha/software-properties/add-update-after-repo-add | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Terry | Needs Fixing | ||
Review via email: mp+150185@code.launchpad.net |
Commit message
Adds a new option -u/--update for updating the source list after a PPA has been added via apt-add-repository
Description of the change
Adds a new option -u/--update for updating the source list after a PPA has been added via apt-add-repository
To post a comment you must log in.
Unmerged revisions
- 837. By Manish Sinha (मनीष सिन्हा)
-
Remove YAY print line
- 836. By Manish Sinha (मनीष सिन्हा)
-
The save should be intended 4 chars left else it will never be executed
- 835. By Manish Sinha (मनीष सिन्हा)
-
Move sp.sourceslist.
save() above options. assume_ update - 834. By Manish Sinha (मनीष सिन्हा)
-
Adds a new option -u/--update for updating the source list after a PPA has been added via apt-add-repository
44 + print("YAY")
Should be dropped.
52 - sp.sourceslist. save() assume_ update: save()
53 + if options.
54 + sp.sourceslist.
Why does the save move inside the if? It seems like without the -u, you should still save.