Helix SwarmHelix Swarm
Loading...

gabor_maghera

gabor_maghera (Gabor Maghera)
  • 1
    Followers
  • 1
    Following
  • 1
    Projects
Full Name
Gabor Maghera
Email address
gmaghera@gmail.com
Followers
charlesa (charlesa)
Following
matt_attaway (Matt Attaway)
Projects
  • VTFS2P4
    • Following
  • Activity
  • Shelves
  • Settings
  • Notifications

  • All Activity
  • Followed Activity
  • Reviews
  • Commits
  • Comments
  • Jobs
9 years agoGabor Maghera commented on job000478 for

I am running into this issue myself. Any pointers would be appreciated.

3 comments
10 years agoGabor Maghera committed change 11572 into
Refreshing branch by copying over Neal's latest.
Add a comment
10 years agoGabor Maghera commented on change 10938 for perforce-software-vtfs2p4:main

Yay! Thanks, Neal.

1 comment
10 years agoGabor Maghera committed change 10797 into
Tweaks with Charles' and Neal's input to get past some TFS environment errors.
Add a comment
10 years agoGabor Maghera committed change 10762 into
Branching to my user area so I can test some minor modifications.
Add a comment
11 years agoGabor Maghera commented on change 8550 (basicuser.cc) for matt-attaway-p4nodeapi:main

Umm, I'm not sure I know what they are. So, I'm probably not using them. 😃

7 comments
11 years agoGabor Maghera commented on change 8550 (basicuser.cc) for matt-attaway-p4nodeapi:main

Way to go! I had method in my program which would replace "\t" with "\t" in the output of p4nodeapi, before supplying it to JSON.parse() to work arou

...

Way to go! I had method in my program which would replace "\t" with "\t" in the output of p4nodeapi, before supplying it to JSON.parse() to work around the issue. Nice to see it fixed. Thank you, @matt_attaway!

 «
7 comments
11 years agoGabor Maghera commented on change 8517 for matt-attaway-p4nodeapi:main

Sure, that sounds good. Publishing it to the npm registry with the new name might be a good idea. I'd be glad to assist if needed.

8 comments
11 years agoGabor Maghera commented on change 8517 for matt-attaway-p4nodeapi:main

Here's a link to that implementation in the npm registry: https://npmjs.org/package/p4

8 comments
11 years agoGabor Maghera commented on change 8517 for matt-attaway-p4nodeapi:main

FYI - there is a fairly simple third party implementation in the npm registry with the same name - it's a wrapper around the p4 command. Npm does not

...

FYI - there is a fairly simple third party implementation in the npm registry with the same name - it's a wrapper around the p4 command. Npm does not distinguish the two, so it sees this third-party implementation as a legit update to yours.

$ npm -q outdated p4
Package Current Wanted Latest Location
p4 0.0.2 0.0.4 0.0.4 p4

 «
8 comments
11 years agoGabor Maghera commented on change 8517 for matt-attaway-p4nodeapi:main

Thank you! 😌

8 comments
15 years agoGabor Maghera committed change 7691 into
Massaged indentations.
Add a comment
15 years agoGabor Maghera committed change 7690 into
Wrapped $cur with double quotes whenever it is passed as an argument in calling a function defined within the file.
 When $cur was null this wa...s causing issues when the argument was referenced within a function.  Passing it as "$cur" will turn it into an empty string. «
Add a comment
15 years agoGabor Maghera committed change 7689 into
Modified syntax for testing whether the $cmd variable is set/unset.
Add a comment
15 years agoGabor Maghera committed change 7688 into
Rollback //guest/gabor_maghera/p4_bash_complete/p4 to revision 1
Add a comment
15 years agoGabor Maghera committed change 7687 into
Changed starting index for adding comands to cmd variable from 1 to 0.
Add a comment
15 years agoGabor Maghera committed change 7686 into
Modified so it works with Mac OS X.
Add a comment
15 years agoGabor Maghera committed change 7685 into
Branching Andrew May's bash p4 autocompletion work into my work area.
 I need to make some tweaks to get working on my MacBook Pro.
Add a comment
15 years agoGabor Maghera committed change 7429 into
Integrating P4Pythonlib into my user area.
Add a comment
17 years agoGabor Maghera committed change 6306 into
Branching cvs2p4 into personal module.
Add a comment
No shelved changes.

Profile Settings

Reset to default

Reviews

Sets default values for how diffs are displayed on all reviews.

Show comments in files
View diffs side-by-side
Show space and newline characters
Ignore whitespace when calculating differences

Time Display

Sets how you would like to display time across Swarm.

Display the time in

Adjust when notifications are sent to you about reviews that you're associated with (as an author, reviewer, project member or moderator).

Email me when:

Reset to default
Check all
I change the state of a review
I am the author, and
a review is requested
files in the review are updated
tests on the review have finished
a vote is cast on a review
the state of the review changes
a review or change is committed
a comment is made on the review or change
a comment on the review or change is updated
Someone likes one of my comments
I am a member, and
a review is requested
a review or change is committed
I am a reviewer, and
files in the review are updated
tests on the review have finished
a vote is cast on a review
the state of the review changes
someone joins or leaves the review
a review or change is committed
a comment is made on the review
a comment on the review is updated
I am a moderator, and
files in the review are updated
tests on the review have finished
a review or change is committed