Skip to content

Conversation

@jafingerhut
Copy link
Collaborator

No description provided.

Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
@jafingerhut jafingerhut changed the title Test CI run with whitespace-only change Change CI to use latest available version of pip to install ptf module Feb 22, 2025
@jafingerhut jafingerhut requested a review from fruffy February 22, 2025 18:28
@jafingerhut
Copy link
Collaborator Author

Without these changes, or something similarly effective, Ubuntu 22.04 tests are failing on the cd ptf ; pip install . step, as you can see on the recent CI failures of a different PR I updated earlier today: #213

I don't know what changed since a few days ago to cause this.


- name: Install
run: |
pip --verbose list
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this necessary?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra debug output isn't necessary, until things do not work, and you do not know why :-)

Not strictly necessary for the build to work. Let me know if you want me to remove it, and I will.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed in commit 5.

Signed-off-by: Andy Fingerhut <andy_fingerhut@alum.wustl.edu>
Copy link
Contributor

@fruffy fruffy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should modernize this setup at some point. I am not sure if modern Python packaging still requires these tools.

@jafingerhut jafingerhut merged commit 77a5ba4 into p4lang:main Feb 23, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants