Skip to content

Use IsPackagedProcess() for package detection in PRI fallback#6500

Open
guimafelipe wants to merge 1 commit into
mainfrom
user/felipeda/use-ispackaged
Open

Use IsPackagedProcess() for package detection in PRI fallback#6500
guimafelipe wants to merge 1 commit into
mainfrom
user/felipeda/use-ispackaged

Conversation

@guimafelipe
Copy link
Copy Markdown
Contributor

@guimafelipe guimafelipe commented May 22, 2026

Use AppModel::Identity::IsPackagedProcess() instead of inferring packaged state from GetDefaultPriFileForCurrentPackage's HRESULT. The old check hr != HRESULT_FROM_WIN32(APPMODEL_ERROR_NO_PACKAGE) silently treats other errors as packaged.

Replace inferring packaged state from GetDefaultPriFileForCurrentPackage's
HRESULT with AppModel::Identity::IsPackagedProcess() which properly
handles GetCurrentPackageFullName errors.
@guimafelipe
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@guimafelipe guimafelipe marked this pull request as ready for review May 22, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants