Sublime-Text-2-Perforce-Plugin icon indicating copy to clipboard operation
Sublime-Text-2-Perforce-Plugin copied to clipboard
verified publisher icon ericmartel

p4v integration

Open mattyork opened this issue 13 years ago • 6 comments

Heyo,

Thanks for starting out this Perforce plugin. Makes life much easier.

This pull adds three commands to the interface: History, Time Lapse, and Revision Graph. All three use p4v to load a single window for the current file. The new command that all three of these use accepts an argument for which p4v window to load, so one could easily add another command that loads a different p4v window by invoking a different argument (e.g. "properties"). I limited the addition to these three because I've found them to be the only useful ones.

It also extracts all values in p4 info into a dictionary. This could replace other functions such as GetUserFromClientspec() and GetClientRoot(), but I did not modify those in order to keep this pull request solely as an addition to existing code instead of a modification.

Let me know if there are any changes I should make. Thanks!

mattyork avatar May 09 '12 22:05 mattyork

Hi Matt!

Thanks a bunch for this submit! I will integrate it tomorrow as I have other tasks planned on this plug in as well.

I'll look into your dictionary usage!

Cheers

Eric

ericmartel avatar May 09 '12 23:05 ericmartel

Bump!

mattyork avatar Aug 02 '12 18:08 mattyork

Oops, I guess "tomorrow" wasn't exactly true :/ I'll look into it again in the following days

Thanks

ericmartel avatar Aug 03 '12 15:08 ericmartel

bump

mattyork avatar Sep 27 '12 20:09 mattyork

Ping, are you going to integrate this change?

ghost avatar May 09 '13 03:05 ghost

I've submit pull request #36 which fixes the merge conflicts and fixed a bug in the p4port value of the generated p4v command.

ptekchand avatar Jun 20 '14 07:06 ptekchand