Skip to content

Add notes about the return codes from lodctr.exe and unlodctr.exe - #8091

Merged
dknappettmsft merged 3 commits into
MicrosoftDocs:mainfrom
Robo210:patch-2
Oct 22, 2025
Merged

Add notes about the return codes from lodctr.exe and unlodctr.exe#8091
dknappettmsft merged 3 commits into
MicrosoftDocs:mainfrom
Robo210:patch-2

Conversation

@Robo210

@Robo210 Robo210 commented Oct 21, 2025

Copy link
Copy Markdown
Member

There has been some confusion about what it means when unlodctr.exe returns 0 but reports and error to stdout.

@prmerger-automator

Copy link
Copy Markdown
Contributor

@Robo210 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 01179fd:

✅ Validation status: passed

File Status Preview URL Details
WindowsServerDocs/administration/windows-commands/lodctr.md ✅Succeeded
WindowsServerDocs/administration/windows-commands/unlodctr.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards
v-dirichards requested a review from Copilot October 21, 2025 21:52
@v-dirichards

Copy link
Copy Markdown
Contributor

@robinharwood, @Xelu86
Can you review the proposed changes?

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team
#assign: @robinharwood, @Xelu86

@prmerger-automator

Copy link
Copy Markdown
Contributor

Users xelu86 are already assigned.

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Oct 21, 2025

Copilot AI 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.

Pull Request Overview

This PR adds clarification about the return codes from lodctr.exe and unlodctr.exe commands. Both utilities return exit code 0 when the command syntax is valid, regardless of whether the operation actually succeeded, which can cause confusion when checking for errors programmatically.

Key Changes:

  • Added a Remarks section to both command documentation files explaining that exit code 0 only indicates valid syntax, not successful execution
  • Fixed heading level for the Remarks section in lodctr.md from #### to ##

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
unlodctr.md Added new Remarks section explaining exit code behavior
lodctr.md Corrected Remarks heading level and added exit code clarification

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread WindowsServerDocs/administration/windows-commands/lodctr.md Outdated
@dknappettmsft
dknappettmsft requested review from Copilot and removed request for Xelu86 October 22, 2025 09:08

Copilot AI 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.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@dknappettmsft

Copy link
Copy Markdown
Contributor

This pull request updates documentation for the lodctr and unlodctr Windows commands to clarify exit code behavior and refresh metadata. The most important changes are grouped below:

Documentation improvements:

  • Added a remark to both lodctr.md and unlodctr.md explaining that an exit code of 0 only indicates correct command line syntax, not necessarily a successful operation, and advises checking command output for errors. [1] [2]

Metadata updates:

  • Updated the ms.date metadata field in both lodctr.md and unlodctr.md to reflect the new documentation date (10/22/2025). [1] [2]

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit fbb02ea:

✅ Validation status: passed

File Status Preview URL Details
WindowsServerDocs/administration/windows-commands/lodctr.md ✅Succeeded
WindowsServerDocs/administration/windows-commands/unlodctr.md ✅Succeeded

For more details, please refer to the build report.

@dknappettmsft

Copy link
Copy Markdown
Contributor

#sign-off

@dknappettmsft
dknappettmsft merged commit d533d45 into MicrosoftDocs:main Oct 22, 2025
2 checks passed
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.

6 participants