p4c icon indicating copy to clipboard operation
p4c copied to clipboard

Rename v1model-newtype.pp to v1model-newtype.p4 and add expected output files?

Open jafingerhut opened this issue 3 years ago • 1 comments

testdata/p4_16_samples/v1model-newtype.pp has no expected output files, and since it has a .pp suffix I believe that the auto-run tests never do anything with it. Keep it? If so, seems best to rename it to end with .p4 and then add expected output files for it.

jafingerhut avatar Oct 22 '22 21:10 jafingerhut

Hmmm. This .pp file appears like it was #include'd by another P4 test program at some point in the past, one named psa-portid-using-newtype2.p4. That file was renamed v1model-portid-using-newtype2.p4, but accidentally (I think) put into the root directory of this repo, then later that file was removed, so now there is no file that #include's v1model-newtype.pp.

@jafingerhut I should probably think about what to do with all of the files containing 'portid-using-newtype' in their names, together with v1model-newtype.pp. I'm not sure what the best way to handle them is, yet, but they should have the same fate.

jafingerhut avatar Oct 23 '22 14:10 jafingerhut