Bugzilla – Full Text Bug Listing
| Summary: | Provide status CLI command | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Technology] JGit | Reporter: | François Rey <eclipse.org> | ||||||||||
| Component: | JGit | Assignee: | François Rey <eclipse.org> | ||||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||||
| Severity: | enhancement | ||||||||||||
| Priority: | P3 | CC: | caniszczyk, fps2, tomasz.zarna | ||||||||||
| Version: | unspecified | Keywords: | noteworthy | ||||||||||
| Target Milestone: | 2.2 | ||||||||||||
| Hardware: | All | ||||||||||||
| OS: | All | ||||||||||||
| Whiteboard: | |||||||||||||
| Bug Depends on: | |||||||||||||
| Bug Blocks: | 365439 | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
François Rey
Created attachment 197354 [details] Patch adding CLI status command This is a first simple implementation. It's been helpful in the diagnostic of bug #347885. I'm the sole writer and owner of this code, and hereby donate it to eclipse under EDL 1.0, the jgit license. Note that my changes to CLIText.properties are not mingled with other lines in order to respect alphabetical order. If that's really important we can certainly apply sorting to my changes. However it's nice to keep them together because some padding alignment needs to be done across some of them. If later on one needs to change a label, other labels will need to be changed accordingly so that padding is respected. I also kept my changes to CLIText.java together instead of sorting. It's less important to keep them together in that file, but did so just because I kept them together in the twin properties file. Finally, it would be nice to also print the current branch like native 'git status'. However being new to git/egit/jgit I wasn't sure of certain corner cases (headless state, etc.) so I preferred not adding that feature. I'm sure someone else can easily add this. Reopening, not sure why this has been save as CLOSED INVALID, I did not mean that. Created attachment 197355 [details]
New version of the patch, previous one had an error
Created attachment 197358 [details]
New patch version - errors in eclipse are gone
Previous patch compiled fine with mvn.
This one just removes unused imports eclipse complained about.
Created Gerrit change http://egit.eclipse.org/r/3682 I've created a new version of this change, the status CLI now: - prints the current branch - prints unmerged count and files Unfortunately, due to some difficulties when migrating my gerrit user to the new eclipse git/gerrit server, pushing it again gave it a new home: https://git.eclipse.org/r/#/c/5765/ ChangeId is still the same though... For reference, old one is still at: http://egit.eclipse.org/r/3682 I'm the sole writer and owner of this code, and hereby donate it to eclipse under EDL 1.0, Created attachment 220816 [details]
ICQ submission
This was merged into master. |