freerdp-nightly: Add version 3.24.3-dev0-1961.5bac12b73

[sitiom]

https://www.freerdp.com/

Notes:

• Command-line output is currently not working for wfreerdp and SDL clients. I have opened a PR on the upstream repo to fix this: fix: console output on Windows client FreeRDP/FreeRDP#12573

• Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>

• I have read the Contributing Guide

Summary by CodeRabbit

• New Features
  • Added a FreeRDP nightly package for Windows (32-bit and 64-bit) with preconfigured extraction and installation of executables.
  • Built-in automated version checking and auto-update support that tracks latest CI builds and derives combined version identifiers for nightly releases.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4a6d1dde-c3f6-40e5-aab3-153c4e785c69

📥 Commits

Reviewing files that changed from the base of the PR and between b48ab28 and 01acbb1.

📒 Files selected for processing (1)

• bucket/freerdp-nightly.json

✅ Files skipped from review due to trivial changes (1)

• bucket/freerdp-nightly.json

---

Walkthrough

Adds a new FreeRDP nightly package manifest that defines artifacts for 32/64-bit Windows, extraction and installed binaries, and an automated checkver/autoupdate routine querying Jenkins and GitHub to construct dynamic version strings.

Changes

Cohort / File(s)	Summary
FreeRDP Nightly Package Manifest bucket/freerdp-nightly.json	New manifest adding a nightly package: metadata (description, homepage, license), architecture-specific CI ZIP URLs and hashes, extract_dir, enumerated bin executables, a checkver routine that queries Jenkins for last successful build and fetches GetProjectVersion.cmake from the corresponding GitHub commit to extract RAW_VERSION_STRING, and an autoupdate template for future artifacts.

Sequence Diagram(s)

sequenceDiagram
    participant Manifest
    participant JenkinsAPI
    participant GitHubRaw
    participant Downloader

    Manifest->>JenkinsAPI: GET /job/.../lastSuccessfulBuild/api/json
    JenkinsAPI-->>Manifest: build number & commit SHA
    Manifest->>GitHubRaw: GET /raw/{commit}/cmake/GetProjectVersion.cmake
    GitHubRaw-->>Manifest: file contents
    Manifest->>Manifest: extract RAW_VERSION_STRING + format version
    Manifest->>Downloader: construct CI ZIP URL(s) and fetch artifacts
    Downloader-->>Manifest: ZIP files (for 32/64-bit)

Loading

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐇 Nights hum with builds and light,

I hop through Jenkins, swift and bright.
SHA crumbs and versions in my paw,
Packaging joy with gentle awe.
FreeRDP dreams in code I saw. ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title follows the conventional format 'freerdp-nightly: Add version 3.24.3-dev0-1961.5bac12b73' and clearly describes the main change: adding a new manifest with a specific version.
Description check	✅ Passed	The description includes the required conventional PR title checkbox (checked), confirmation of reading the Contributing Guide (checked), and relevant context about upstream issues. However, it lacks the optional issue reference (Closes/Relates #XXXX) section.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@bucket/freerdp-nightly.json`:
- Around line 30-33: The error handling in the checkver PowerShell snippet uses
"break" after reporting an error for missing $sha and for a failed
$versionMatch, which can silently terminate the runspace; update both
occurrences to use "throw" so the error is propagated (i.e., replace the
post-error "break" statements following the $sha check and the
$versionMatch.Success check that parses GetProjectVersion.cmake into
$cmake/$versionMatch with "throw" to properly raise exceptions).

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 75610f81-1567-4467-88df-64b169242f0a

📥 Commits

Reviewing files that changed from the base of the PR and between 1e781e6 and b48ab28.

📒 Files selected for processing (1)

• bucket/freerdp-nightly.json

[sitiom]

/verify

[github-actions]

All changes look good.

Wait for review from human collaborators.

freerdp-nightly

• Lint
• Description
• License
• Hashes
• Checkver
• Autoupdate

Check the full log for details.

[sitiom]

/verify

[github-actions]

All changes look good.

Wait for review from human collaborators.

freerdp-nightly

• Lint
• Description
• License
• Hashes
• Checkver
• Autoupdate

Check the full log for details.

[z-Fng]

Thanks for your contribution!