Skip to content

Conversation

@BillyONeal
Copy link
Member

@BillyONeal BillyONeal added the category:infrastructure Pertaining to the CI/Testing infrastrucutre label Oct 10, 2025
@Neumann-A
Copy link
Contributor

Neumann-A commented Oct 11, 2025

@BillyONeal
Fix file lists
0236e6a

File list seems still to be broken (at least x64-windows-static is empty.)

@Neumann-A
Copy link
Contributor

probably blocked by #47764. I assume dlib is the reason why static builds somehow expect MKL to be available however I have not yet have a trace of the configure log with that error.

@BillyONeal BillyONeal merged commit 04dc291 into microsoft:master Oct 13, 2025
15 of 18 checks passed
@BillyONeal BillyONeal deleted the release-2025-10-10 branch October 13, 2025 18:27
@ras0219-msft
Copy link
Contributor

File list seems still to be broken (at least x64-windows-static is empty.)

I think this is intended behavior -- the new PR system only generates lists for directly modified ports (e.g. git diff -- ports/). CI builds should still generate lists for all ports.

@Neumann-A
Copy link
Contributor

the new PR system only generates lists for directly modified ports

Then you can never check what actually was installed for a given set of ports. I wanted to look at the list because i wanted to see if a port has been installed by a run, if the files where really missing or if the libs simply had a different name than expected.
If you only get the list for touched ports you cannot extra these infos from the file list.

@dg0yt
Copy link
Contributor

dg0yt commented Oct 15, 2025

Since this update, we see feature tests failing in the middle, e.g. #47812.

@dg0yt
Copy link
Contributor

dg0yt commented Oct 15, 2025

... and this one: #47803

@dg0yt
Copy link
Contributor

dg0yt commented Oct 15, 2025

... and #47815, which is really just triggering a rebuild.
🏠 🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:infrastructure Pertaining to the CI/Testing infrastrucutre

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants