Jump to content


P4V reconcile inconsistency

P4V reconcile

  • Please log in to reply
12 replies to this topic

#1 mister_wilson

mister_wilson

    Advanced Member

  • Members
  • PipPipPip
  • 109 posts
  • LocationMontreal, Quebec

Posted 07 February 2019 - 04:46 PM

Hi,

I am currently using P4V version Rev. P4V/NTX64/2018.4/1753667 on Windows 10 and my server version is P4D/LINUX26X86_64/2016.2/1568552 (2017/09/20)

I seem to be experiencing a strange inconsistency with the p4 reconcile operation.
When I select the Reconcile Offline Work from P4V the operation works as expected (see attached screenshots 1, 2, and 3)

However when i try to run the same command from the command line (after selecting Open Command Window Here) nothing get reconciled.

i.e.:

p4 reconcile -c 1205131 -a -d -e -f //mydepot/SubmitAssistant/Default/...
//mydepot/SubmitAssistant/Default/... - no file(s) to reconcile.

Would anyone have any idea why the operation doesn't work via the command line?

Let me know if you need any additional information.

Thanks,

-Jonathan Wilson

Attached Thumbnails

  • reconcile_example_1.png
  • reconcile_example_2.png
  • reconcile_example_3.png


#2 Sambwise

Sambwise

    Advanced Member

  • Members
  • PipPipPip
  • 782 posts

Posted 07 February 2019 - 06:29 PM

What does "p4 opened" show from that same command prompt?

#3 mister_wilson

mister_wilson

    Advanced Member

  • Members
  • PipPipPip
  • 109 posts
  • LocationMontreal, Quebec

Posted 07 February 2019 - 06:45 PM

Hi Sam,

The commands returns:
File(s) not opened on this client.


#4 Sambwise

Sambwise

    Advanced Member

  • Members
  • PipPipPip
  • 782 posts

Posted 07 February 2019 - 06:50 PM

Hm, I was thinking that maybe the files had already been opened.  Okay, what about:


p4 have //mydepot/SubmitAssistant/Default/Restrictions.config
p4 diff -f //mydepot/SubmitAssistant/Default/Restrictions.config
p4 reconcile
//mydepot/SubmitAssistant/Default/Restrictions.config
p4 opened //mydepot/SubmitAssistant/Default/Restrictions.config

The "p4 diff" command will force a diff of that file.  If it shows no differences, then reconcile should not open it.  If it does show differences, then reconcile should open it, and a subsequent "p4 opened" should show it as open for edit.  If that "p4 have" shows nothing, it means the file isn't synced to your workspace and so there'll be nothing to diff against.

#5 mister_wilson

mister_wilson

    Advanced Member

  • Members
  • PipPipPip
  • 109 posts
  • LocationMontreal, Quebec

Posted 07 February 2019 - 07:00 PM

Hi Sam,

Here are the results of the 4 operations:

f:\mydepot\SubmitAssistant\Default>p4 have //mydepot/SubmitAssistant/Default/Restrictions.config
//mydepot/SubmitAssistant/Default/Restrictions.config#44 - F:\mydepot\SubmitAssistant\Default\Restrictions.config


f:\mydepot\SubmitAssistant\Default>p4 diff -f //mydepot/SubmitAssistant/Default/Restrictions.config
==== //mydepot/SubmitAssistant/Default/Restrictions.config#44 - F:\mydepot\SubmitAssistant\Default\Restrictions.config ====
1c1
< <?xml version="1.0" encoding="utf-8" ?>
---
> ´╗┐<?xml version="1.0" encoding="utf-8"?>
4d3
<
11,34c10,31
<
< <ReviewerRestriction Ports="Code">
< <IncludeFile Pattern="//hl3/dev/main/code/AgentGroupImpl/ProtectorGroup/" Users="gprovost"/>
< <IncludeFile Pattern="//hl3/dev/main/code/DriverAI/Zones/" Users="gprovost"/>
< <IncludeFile Pattern="//hl3/dev/main/code/GameMoveIdsData/" Users="gprovost"/>
< <IncludeFile Pattern="//hl3/dev/main/code/GameplayZones/" Users="gprovost"/>
< <IncludeFile Pattern="//hl3/dev/main/code/GameplayZonesData/" Users="gprovost"/>
< <IncludeFile Pattern="//hl3/dev/main/code/Sharpmake/SharpmakeFacade/" Users="hross"/>
< <IncludeFile Pattern="//hl3/dev/main/code/Zones/" Users="gprovost"/>
< <IncludeFile Pattern="//hl3/dev/main/external/technology-group/dtest/" Users="mdemers"/>
< <IncludeFile Pattern="//hl3/dev/main/bin/tools/dtest/deploy/" Users="mdemers"/>
< <IncludeFile Pattern="//hl3/dev/main/code/AnimShared/" Users="fzimmer"/>
< <IncludeFile Pattern="//hl3/dev/main/code/IKToolkit/" Users="fzimmer"/>
< <IncludeFile Pattern="//hl3/dev/main/code/ReinforcementData/" Users="vguesdon"/>
< <IncludeFile Pattern="//hl3/dev/main/code/ReinforcementServices/" Users="vguesdon"/>
< <IncludeFile Pattern="//hl3/dev/main/code/SpawningRealisationServicesData/" Users="vguesdon"/>
< <IncludeFile Pattern="//hl3/dev/main/code/SpawningRealisationServices/" Users="vguesdon"/>
< <IncludeFile Pattern="//hl3/dev/main/code/BarkServices/" Users="vguesdon"/>
< <IncludeFile Pattern="//hl3/dev/main/code/BarkServicesData/" Users="vguesdon"/>
< <IncludeFile Pattern="//hl3/dev/main/code/WorldAI/" Users="vguesdon"/>
< <IncludeFile Pattern="//hl3/dev/main/code/SpawningRealisation/" Users="vguesdon"/>
< <IncludeFile Pattern="//hl3/dev/main/tools/SubmitAssistant/" Users="jsenechal"/>
< </ReviewerRestriction>
<
---
> <ReviewerRestriction Ports="Code">
> <IncludeFile Pattern="//hl3/dev/main/code/AgentGroupImpl/ProtectorGroup/" Users="gprovost" />
> <IncludeFile Pattern="//hl3/dev/main/code/DriverAI/Zones/" Users="gprovost" />
> <IncludeFile Pattern="//hl3/dev/main/code/GameMoveIdsData/" Users="gprovost" />
> <IncludeFile Pattern="//hl3/dev/main/code/GameplayZones/" Users="gprovost" />
> <IncludeFile Pattern="//hl3/dev/main/code/GameplayZonesData/" Users="gprovost" />
> <IncludeFile Pattern="//hl3/dev/main/code/Sharpmake/SharpmakeFacade/" Users="hross" />
> <IncludeFile Pattern="//hl3/dev/main/code/Zones/" Users="gprovost" />
> <IncludeFile Pattern="//hl3/dev/main/external/technology-group/dtest/" Users="mdemers" />
> <IncludeFile Pattern="//hl3/dev/main/bin/tools/dtest/deploy/" Users="mdemers" />
> <IncludeFile Pattern="//hl3/dev/main/code/AnimShared/" Users="fzimmer" />
> <IncludeFile Pattern="//hl3/dev/main/code/IKToolkit/" Users="fzimmer" />
> <IncludeFile Pattern="//hl3/dev/main/code/ReinforcementData/" Users="vguesdon" />
> <IncludeFile Pattern="//hl3/dev/main/code/ReinforcementServices/" Users="vguesdon" />
> <IncludeFile Pattern="//hl3/dev/main/code/SpawningRealisationServicesData/" Users="vguesdon" />
> <IncludeFile Pattern="//hl3/dev/main/code/SpawningRealisationServices/" Users="vguesdon" />
> <IncludeFile Pattern="//hl3/dev/main/code/BarkServices/" Users="vguesdon" />
> <IncludeFile Pattern="//hl3/dev/main/code/BarkServicesData/" Users="vguesdon" />
> <IncludeFile Pattern="//hl3/dev/main/code/WorldAI/" Users="vguesdon" />
> <IncludeFile Pattern="//hl3/dev/main/code/SpawningRealisation/" Users="vguesdon" />
> <IncludeFile Pattern="//hl3/dev/main/tools/SubmitAssistant/" Users="jsenechal" />
> </ReviewerRestriction>
36c33
< <IncludeFile Pattern="//hl3/dev/main/"/>
---
> <IncludeFile Pattern="//hl3/dev/main/" />
38,40c35,36
<
< <ReviewerRestriction Ports="Code" Sites="Globant" Groups="CodeReviewers-GLO">
< <IncludeFile Pattern="//hl3/dev/main/"/>
---
> <ReviewerRestriction Ports="Code" Sites="Globant" Groups="CodeReviewers-GLO">
> <IncludeFile Pattern="//hl3/dev/main/" />
42d37
<
45,46c40,42
< </Restriction>
< <Restriction Ports="Code" Sites="Globant" Groups="CodeApprovers-GLO">
---
> <IncludeFile Pattern="//hl3/dev/testfakedepot/" />
> </Restriction>
> <Restriction Ports="Code" Sites="Globant" Groups="CodeApprovers-GLO">
48,49c44
< </Restriction>
<
---
> </Restriction>
53c48
< <Restriction Ports="Data" Sites="Globant" Groups="DataApprovers-GLO">
---
> <Restriction Ports="Data" Sites="Globant" Groups="DataApprovers-GLO">
56,57c51
<
< <!-- Locker to everyone, nobody can submit files under this path-->
---
> <!-- Locker to everyone, nobody can submit files under this path-->
61d54
<
63d55
<
66c58
< <ErrorRestriction Ports="Code" Groups="CodeApprovers-GLO" Sites="Globant" />
---
> <ErrorRestriction Ports="Code" Groups="CodeApprovers-GLO" Sites="Globant" />
74d65
<
79d69
<
82,86c72,75
< <!-- Codevs cannot access our mailing server, so we have to plainly list all users here...-->
< <Group Name="CodeReviewers-GLO" Users ="gcarvalhocunha,jbarrois,jlee5,fbessette,gprovost,yzhou2,yjoly,fzimmer,ncolotouros,eallard,hross,ytriqueneaux,gdecesare,mlabeyrie,marsenault,mafaucher,aaudettegenier,olamothepenelle,jhope,mdemers,naudet,kstasik,ysimard,vguesdon,dforget,sleclerc,jdaignault,sgirard,jegginton,vpuech,npowell,jfroy3,vchartrand2,carbez,david.punset,florian.gimat,carlos.caicedo,swhitwill,jaudran,jpotter,pochartrand,mgabriac,legger,mlheureux,mvachon,phil.palmer,jmtannous,fchartrand,jherbin,nbeaufils,jfouellet,jfracine,stremblay2,jfmarquis,fferland,dlynch,hsimard2,ehebert,mgruvman,fboivin,mlaforce,pzink,jbolduc,ldecarufel"/>
< <Group Name="CodeApprovers-GLO" Users ="naudet,ysimard,gfrechette,sgirard,aricher,jfroy3,david.punset,pochartrand,jmtannous,nbeaufils,dtherriault,hsimard2,fboivin,pzink"/>
< <Group Name="DataApprovers-GLO" Users ="ysimard,gfrechette,sgirard,aricher,madeslongchamps,jfroy3,scharbonneau2,nmainville,souellet,nevleduc,pochartrand,ogirard,broy,alex.mayer,legger,jfdubois,dtherriault,hsimard2,pzink,mhebert,ppelletier,mhuet,garcand"/>
<
---
> <!-- Codevs cannot access our mailing server, so we have to plainly list all users here...-->
> <Group Name="CodeReviewers-GLO" Users="gcarvalhocunha,jbarrois,jlee5,fbessette,gprovost,yzhou2,yjoly,fzimmer,ncolotouros,eallard,hross,ytriqueneaux,gdecesare,mlabeyrie,marsenault,mafaucher,aaudettegenier,olamothepenelle,jhope,mdemers,naudet,kstasik,ysimard,vguesdon,dforget,sleclerc,jdaignault,sgirard,jegginton,vpuech,npowell,jfroy3,vchartrand2,carbez,david.punset,florian.gimat,carlos.caicedo,swhitwill,jaudran,jpotter,pochartrand,mgabriac,legger,mlheureux,mvachon,phil.palmer,jmtannous,fchartrand,jherbin,nbeaufils,jfouellet,jfracine,stremblay2,jfmarquis,fferland,dlynch,hsimard2,ehebert,mgruvman,fboivin,mlaforce,pzink,jbolduc,ldecarufel" />
> <Group Name="CodeApprovers-GLO" Users="naudet,ysimard,gfrechette,sgirard,aricher,jfroy3,david.punset,pochartrand,jmtannous,nbeaufils,dtherriault,hsimard2,fboivin,pzink" />
> <Group Name="DataApprovers-GLO" Users="ysimard,gfrechette,sgirard,aricher,madeslongchamps,jfroy3,scharbonneau2,nmainville,souellet,nevleduc,pochartrand,ogirard,broy,alex.mayer,legger,jfdubois,dtherriault,hsimard2,pzink,mhebert,ppelletier,mhuet,garcand" />
88c77
< <Group Name="Approvers" Users="pzink,jfroy3,sgirard"/>
---
> <Group Name="Approvers" Users="pzink,jfroy3,sgirard" />
92d80
<
95,99c83,87
< <Site Name="Globant" Prefix="GLO" />
< <Site Name="Montreal" Prefix="MTL"/>
< <Site Name="Paris" Prefix="MRC"/>
< <Site Name="Bucarest" Prefix="BUC"/>
< <Site Name="Newcastle" Prefix="NCT"/>
---
> <Site Name="Globant" Prefix="GLO" />
> <Site Name="Montreal" Prefix="MTL" />
> <Site Name="Paris" Prefix="MRC" />
> <Site Name="Bucarest" Prefix="BUC" />
> <Site Name="Newcastle" Prefix="NCT" />
102c90
< <Site Name="CUC" Prefix="CWC"/>
---
> <Site Name="CUC" Prefix="CWC" />
106,108c94,96
< <Port Name="Code" PortNumber="2326"/>
< <Port Name="Data" PortNumber="3626"/>
< <Port Name="Sound" PortNumber="3636"/>
---
> <Port Name="Code" PortNumber="2326" />
> <Port Name="Data" PortNumber="3626" />
> <Port Name="Sound" PortNumber="3636" />


f:\mydepot\SubmitAssistant\Default>p4 reconcile //mydepot/SubmitAssistant/Default/Restrictions.config
//mydepot/SubmitAssistant/Default/Restrictions.config - no file(s) to reconcile.


f:\mydepot\SubmitAssistant\Default>p4 opened //mydepot/SubmitAssistant/Default/Restrictions.config
//mydepot/SubmitAssistant/Default/Restrictions.config - file(s) not opened on this client.

This one has me scratching my head as P4V doesn't seem to have a problem with this and I opened my command line window from the P4V environment. :blink: :unsure: :wacko:

#6 Sambwise

Sambwise

    Advanced Member

  • Members
  • PipPipPip
  • 782 posts

Posted 07 February 2019 - 07:21 PM

Are the version numbers reported by "p4 -V" and P4V's "About" dialog significantly different?  The most likely thing I can think of is that there's a client-side bug in your version of P4 that isn't present in your P4V's client library.

The one other thing I'd be curious about would be "p4 diff -se" (which will do the comparison more the way that reconcile would do it, i.e. via checksum) but I can't think of any way that the checksum on the server would just happen to be wrong in such a way that it matches what's in your workspace, nor would that explain why it works in P4V but not P4.

#7 mister_wilson

mister_wilson

    Advanced Member

  • Members
  • PipPipPip
  • 109 posts
  • LocationMontreal, Quebec

Posted 07 February 2019 - 07:41 PM

I've attached my P4V's about dialog. The versions indeed appear to be different.

f:\>p4 -V
Perforce - The Fast Software Configuration Management System.
Copyright 1995-2019 Perforce Software. All rights reserved.
This product includes software developed by the OpenSSL Project
for use in the OpenSSL Toolkit (http://www.openssl.org/)
Version of OpenSSL Libraries: OpenSSL 1.0.2q 20 Nov 2018
See 'p4 help [ -l ] legal' for additional license information on
these licenses and others.
Extensions/scripting support built-in.
Parallel sync threading built-in.
Rev. P4/NTX64/2018.2/1751184 (2019/01/21).


The result from p4 diff -se
f:\mydepot\SubmitAssistant\Default>p4 diff -se //mydepot/SubmitAssistant/Default/Restrictions.config
F:\mydepot\SubmitAssistant\Default\Restrictions.config

Attached Thumbnails

  • about_p4v.png


#8 mister_wilson

mister_wilson

    Advanced Member

  • Members
  • PipPipPip
  • 109 posts
  • LocationMontreal, Quebec

Posted 07 February 2019 - 08:51 PM

BTW I was running some more tests and i seem to only have the issue with this particular depot.
I ran reconcile operations on other depots (streamed, non-stream) and the reconcile operation works fine through P4V and the command line.

The depot in question appears to have been created correctly:

# A Perforce Depot Specification.
#
#  Depot:	   The name of the depot.
#  Owner:	   The user who created this depot.
#  Date:		The date this specification was last modified.
#  Description: A short description of the depot (optional).
#  Type:		Whether the depot is 'local', 'remote',
#			   'stream', 'spec', 'archive', 'tangent',
#			   or 'unload'.  Default is 'local'.
#  Address:	 Connection address (remote depots only).
#  Suffix:	  Suffix for all saved specs (spec depot only).
#  StreamDepth: Depth for streams in this depot (stream depots only).
#  Map:		 Path translation information (must have ... in it).
#  SpecMap:	 For spec depot, which specs should be recorded (optional).
#
# Use 'p4 help depot' to see more about depot forms.

Depot: mydepot

Owner: jwilson

Date: 2017/11/27 14:26:39

Description:
Created by jwilson.

Type: local

Map: mydepot/...


#9 mister_wilson

mister_wilson

    Advanced Member

  • Members
  • PipPipPip
  • 109 posts
  • LocationMontreal, Quebec

Posted 07 February 2019 - 09:13 PM

Hi Sam,

I found the issue, it was just a silly permission problem.
The user account that i was using only had read access to the depot.
I can't believe i didn't figure it out sooner.

Sorry for the inconveniences,

-JW

#10 mister_wilson

mister_wilson

    Advanced Member

  • Members
  • PipPipPip
  • 109 posts
  • LocationMontreal, Quebec

Posted 07 February 2019 - 09:14 PM

I probably would have seen it if I continued with the reconcile operation in P4V instead of the stopping at the preview window :angry:

#11 Sambwise

Sambwise

    Advanced Member

  • Members
  • PipPipPip
  • 782 posts

Posted 07 February 2019 - 09:56 PM

When you said it was only a problem with one depot I thought "oh, maybe he doesn't have 'open' access to that depot?" but you got there on your own.  :D

Seems like a bug in reconcile that it tells you "no files to reconcile" rather than "you don't have permission" in that situation.  It should be a simple matter for it to check the "open" permission map and see that it's empty...

#12 mister_wilson

mister_wilson

    Advanced Member

  • Members
  • PipPipPip
  • 109 posts
  • LocationMontreal, Quebec

Posted 08 February 2019 - 09:56 PM

That was exactly it. The account i was using only had 'read' access instead of open.
I was thinking the same thing about the message being returned by the reconcile operation, it would have been nice if the message was clearer.
(lol even a message calling me an 'idiot' for not checking permissions would have been tolerable ^_^ )

Oh BTW I should point out that when i add the preview option '-n' in the command line, it outputs the message that the file is opened for edit. Leading us to believe that the command would have worked without the preview option.

#13 Sambwise

Sambwise

    Advanced Member

  • Members
  • PipPipPip
  • 782 posts

Posted 08 February 2019 - 09:57 PM

View Postmister_wilson, on 08 February 2019 - 09:56 PM, said:

Oh BTW I should point out that when i add the preview option '-n' in the command line, it outputs the message that the file is opened for edit. Leading us to believe that the command would have worked without the preview option.

That also sounds like a bug...





0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users