Skip to content

Prerequisite 4 for #105403 (src/coreclr/tools/superpmi) - #117022

Closed
gwr wants to merge 2 commits into
dotnet:mainfrom
gwr:illumos4d
Closed

Prerequisite 4 for #105403 (src/coreclr/tools/superpmi)#117022
gwr wants to merge 2 commits into
dotnet:mainfrom
gwr:illumos4d

Conversation

@gwr

@gwrgwr commented Jun 25, 2025

Copy link
Copy Markdown
Contributor

Prerequisite 4 for #105403
Changes under src/coreclr/tools/superpmi

am11and others added 2 commits June 25, 2025 10:05
 /runtime/src/coreclr/tools/superpmi/mcs/verbmerge.cpp: In static member function 'static bool verbMerge::DirectoryFilterDirectories(FilterArgType*)':
/runtime/src/coreclr/tools/superpmi/mcs/verbmerge.cpp:188:19: error: 'verbMerge::FilterArgType' {aka 'struct dirent'} has no member named 'd_type'
188 | if (findData->d_type == DT_DIR)
| ^~~~~~
and similar a few other places in this file
Let FilterArgType use struct FindData to simplify filters.
@github-actionsgithub-actionsBot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jun 25, 2025
@dotnet-policy-servicedotnet-policy-serviceBot added the community-contribution Indicates that the PR has been added by a community member label Jun 25, 2025
@gwrgwr changed the title Illumos4dPrerequisite 4 for #105403 (src/coreclr/tools/superpmi)Jun 25, 2025
@gwr

gwr commented Jun 25, 2025

Copy link
Copy Markdown
ContributorAuthor

See #117023

@gwrgwr closed this Jun 25, 2025
@gwr
gwr deleted the illumos4d branch June 25, 2025 17:51
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Jul 26, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community-contributionIndicates that the PR has been added by a community memberneeds-area-labelAn area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@gwr@am11