Menu

#82 p4file doesn't exist

V5.2.1
closed-fixed
Perforce (2)
5
2021-12-04
2021-12-03
No

When using perforce, an error with a p4file doesn't exist file.
I believe the $p4file var should be replaced with $fn

Discussion

  • Deepak Yadav

    Deepak Yadav - 2021-12-04

    This may be the fix :

    1774c1775
    <         set finfo(${A}tmp,$ndx) "$cmd print -q \"${p4file}$rev\""
    ---
    >         set finfo(${A}tmp,$ndx) "$cmd print -q \"${fn}$rev\""
    
     
  • michael-m

    michael-m - 2021-12-04
    • status: open --> closed-fixed
    • assigned_to: michael-m
     
  • michael-m

    michael-m - 2021-12-04

    Duh. My bad.
    You are correct - the change you indicate is exactly the fix and will be made as of the next release (obviously you need not wait to fix your own copy). Sadly I dont have an installation of every SCM known to man (or more specifically TkDiff) so oversights like this are an unfortunate fact of life (you cant test what you cant invoke).
    Thank you for not only reporting but researching the issue, I gratefully accept all assistance that comes my way.

     

Log in to post a comment.

Auth0 Logo