P4Transfer needs -f option on reconcile to handle depot paths containing wildcards.
The p4 reconcile command in P4Transfer.py needs this option to make transfers more reliable.
The -f is required with p4 add (and thus p4 rec) to emphasize that depot paths being added contain "funky" wildcard characters, such as %, @, #, or *. Such names have special meaning in P4, and thus are disallowed by default. But because can override the default simply by adding -f, it seems reasonable that P4Transfer should include that option by default to avoid a predictable and preventable transfer failure.
Elsewhere in the code, it is clear that the -f had been added previously, e.g.:
self.p4cmd('add', '-ft', f.type, f.fixedLocalFile)
This request is to add the -f to the p4 reconcile command as well, thus closing a gap.
P4Transfer needs -f option on reconcile to handle depot paths containing wildcards.
The
p4 reconcilecommand in P4Transfer.py needs this option to make transfers more reliable.The
-fis required withp4 add(and thusp4 rec) to emphasize that depot paths being added contain "funky" wildcard characters, such as %, @, #, or *. Such names have special meaning in P4, and thus are disallowed by default. But because can override the default simply by adding-f, it seems reasonable that P4Transfer should include that option by default to avoid a predictable and preventable transfer failure.Elsewhere in the code, it is clear that the
-fhad been added previously, e.g.:This request is to add the
-fto thep4 reconcilecommand as well, thus closing a gap.