Skip to content

Remove unnecessary dependency on mergeNativeLibsTask for customSymbolsDirectory and Unity#2133

Merged
priettt merged 1 commit intomainfrom
priettt/fixCustomSymbolsDirDependencies
Apr 28, 2025
Merged

Remove unnecessary dependency on mergeNativeLibsTask for customSymbolsDirectory and Unity#2133
priettt merged 1 commit intomainfrom
priettt/fixCustomSymbolsDirDependencies

Conversation

@priettt
Copy link
Contributor

@priettt priettt commented Apr 28, 2025

Goal

When using mergeNativeLibsTask.flatMap at the start of the shared object files provider, we were enforcing a dependency on the mergeNativeLibs task for Unity projects and projects that set up a custom symbols directory.

While I think most Unity projects will have that task, I'm not sure if 100% of them will. Also, this would've been an unintended change from our last version.

@priettt priettt requested a review from a team as a code owner April 28, 2025 15:16
@priettt priettt force-pushed the priettt/fixCustomSymbolsDirDependencies branch from 0cf0a2c to d3792d3 Compare April 28, 2025 15:29
@priettt priettt force-pushed the priettt/fixUnitySharedObjectFileProvider branch from 79aa685 to 0aec9e5 Compare April 28, 2025 16:37
Base automatically changed from priettt/fixUnitySharedObjectFileProvider to main April 28, 2025 17:38
@priettt priettt force-pushed the priettt/fixCustomSymbolsDirDependencies branch from d3792d3 to 81eab56 Compare April 28, 2025 17:41
@github-actions
Copy link
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@priettt priettt merged commit 729cdf7 into main Apr 28, 2025
11 checks passed
@priettt priettt deleted the priettt/fixCustomSymbolsDirDependencies branch April 28, 2025 19:05
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