Skip to content

Fix the addition of assemblies and sql scripts to module build - #7098

Merged
valadas merged 3 commits into
dnnsoftware:developfrom
donker:fix-build-20260309
Mar 9, 2026
Merged

Fix the addition of assemblies and sql scripts to module build#7098
valadas merged 3 commits into
dnnsoftware:developfrom
donker:fix-build-20260309

Conversation

@donker

@donker donker commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

The recent changes to the module packaging script meant that some files were not being packaged. This PR aims to fix that.

@donker donker added this to the 10.2.4 milestone Mar 9, 2026
@donker
donker requested review from bdukes and valadas March 9, 2026 08:41
@donker donker self-assigned this Mar 9, 2026

@bdukes bdukes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This added a dll to the root of the package, it didn't put it in a bin folder in the zip

@bdukes
bdukes force-pushed the fix-build-20260309 branch from 201f3fe to ce266ec Compare March 9, 2026 16:56
@bdukes

bdukes commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

I've pushed a fix which accounts for the path coming from sourceFileName, in addition to path. Thanks for tracking down the source of the issue. I can't find where the inline task is any different than the compiled task, but somehow that RecursiveDir was being correctly populated before.

@bdukes
bdukes requested a review from mitchelsellers March 9, 2026 17:06
@bdukes

bdukes commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

One more fix coming, it's picking up files marked with action="UnRegister", which obviously shouldn't be included in the package. (And, again, I feel like I'm missing something, since this wasn't an issue before).

@bdukes
bdukes force-pushed the fix-build-20260309 branch from 85d69b7 to 780e9aa Compare March 9, 2026 17:59
@valadas
valadas merged commit 1464f45 into dnnsoftware:develop Mar 9, 2026
3 checks passed
@bdukes bdukes mentioned this pull request Mar 10, 2026
@valadas valadas modified the milestones: 10.2.4, 10.3.0 Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants