p4 revert files not working with -a option using p4api.net method

Question

Trying to revert files in a changelist using -a option in p4api.net method. It used to work for me but now am getting exception with below message.

Also p4 edit and p4 revert with -c option works fine but p4 revert with -a option throws below exception. I don't know why its picking below test project location in p4 workspace location.

Exception:

Path 'd:\cftt\Dev\source\BRF\BRF.Business.Test\bin\Debug\19402547' is not under client's root 'D:\p4'.

// to open files for edit in a given changelist at certain loc with particular file format 
          public IList<FileSpec> EditChangeList(string clNumber, string fileFormat, string destinationPath)
    {
        try
        {
            var rep = Connect();
            var opts = new Options(ChangeCmdFlags.None, ChangeListType.None);
            opts["-c"] = clNumber;
            var fs = new FileSpec(new DepotPath(destinationPath + "/..." + fileFormat));
            IList<FileSpec> editedFileSpec = rep.Connection.Client.EditFiles(new List<FileSpec> {fs}, opts);
            return editedFileSpec;
        }
        catch (Exception exc)
        {
            Logger.LogError(exc.Message);
            throw;
        }
    }

    // to revert files in a changelist that are unchanged using -a option
           public IList<FileSpec> RevertChangeList(string clNumber, string destinationPath)
    {
        try
        {
            var rep = Connect();
            var opts = new Options(ChangeCmdFlags.None, ChangeListType.None);
            opts["-a"] = clNumber;
            var fs = new FileSpec(new DepotPath(destinationPath + "/..."));
            IList<FileSpec> revertedFiles = rep.Connection.Client.RevertFiles(new List<FileSpec> {fs}, opts);
            return revertedFiles;
        }
        catch (Exception exc)
        {
            Logger.LogError(exc.Message);
            throw;
        }
    }

Answer 1

The revert command takes several different types of flags and arguments:

• flags like -a, -n, and -w, which don't have any value, but are just the flag
• the -c flag, which takes the changelist number as a value for the flag
• the files to be reverted, which are specified as arguments after all the flags (you can specify multiple file arguments)

In your command, you ran:

p4 revert -a 19402547 d:\p4\sw\dev\gpu_drv\chips_a\diag\mods\sim\resources/...

Since the -a flag doesn't take a value, the revert command saw this as:

• the -a flag
• a file argument: 19402547
• a file argument: d:\p4\sw\dev\gpu_drv\chips_a\diag\mods\sim\resources/...

Since the file argument 19402547 did not have a directory name in front of it, the revert command interpreted it as a relative file name, relative to your current directory, and looked for a file named 'd:\cftt\Dev\source\BRF\BRF.Business.Test\bin\Debug\19402547' , which indeed is not under your client's Root directory, and so isn't a file that Perforce is capable of reverting.

The right approach here is either to omit the changelist number entirely, or to specify it as the value of the -c flag, depending on whether you wish to revert all open files in the sim\resources directory, or whether you only wish to revert those open files in that directory which are open in changelist 19402547.

That is, both:

p4 revert -a d:\p4\sw\dev\gpu_drv\chips_a\diag\mods\sim\resources/...

and

p4 revert -a -c 19402547 d:\p4\sw\dev\gpu_drv\chips_a\diag\mods\sim\resources/...

are reasonable commands, and the right one to pick depends on precisely which behavior you want.