Skip to content

Conversation

@haussmann
Copy link
Contributor

@haussmann haussmann commented Dec 18, 2025

I beleive IDA Free should be deprecated, because:

  • Newer versions require registration (source) and the generation of a system-sepcific (free) license key
  • IDA 8.4.240527 (SP2) was released in May 27, 2024 (source). Since then there have been multiple newer versions released. Newest version is 9.2 from September 2025 (source)
  • Checkver and Autoupdate are broken

Relates to #16379

Summary by CodeRabbit

  • Chores
    • Added deprecation notice indicating IDA Free versions from December 18, 2025 onward require user registration and no longer support silent installation.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 18, 2025

Walkthrough

A deprecation entry is added to the IDA Free configuration file, noting that new versions require user registration and cannot be installed silently. No functional changes to existing configuration fields.

Changes

Cohort / File(s) Change Summary
Deprecation Update
deprecated/ida-free.json
Added deprecation message under "##" key documenting registration requirement and silent install limitation as of 2025-12-18

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 The IDA's changed its registration way,
No silent installs brighten up the day,
Yet worry not, dear code we'll mark with care,
A deprecation note floats gently in the air,
The registry calls, and we document the dance! 📝✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'ida-free: Deprecate manifest' clearly and specifically summarizes the main change in the changeset - marking the ida-free manifest as deprecated.
Description check ✅ Passed The description provides clear reasoning for deprecation with linked sources, addresses the required conventional PR title format and contributing guide checklist items, and relates to an existing issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7e7295f and 85b2802.

📒 Files selected for processing (1)
  • deprecated/ida-free.json (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: WindowsPowerShell
🔇 Additional comments (1)
deprecated/ida-free.json (1)

2-4: LGTM! Clear and accurate deprecation notice.

The deprecation message correctly explains the reason (registration requirement) and consequence (silent install no longer possible). The date and rationale align with the PR objectives.


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

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

@haussmann haussmann marked this pull request as draft December 20, 2025 23:01
@haussmann haussmann marked this pull request as ready for review December 22, 2025 09:27
z-Fng added a commit that referenced this pull request Dec 27, 2025
@z-Fng
Copy link
Member

z-Fng commented Dec 27, 2025

We generally don't deprecate a package unless the package cannot be obtained at all.

Please refer to the discussion in #16379 (comment). Closing this as it has been fixed in a later commit (4938450).

@z-Fng z-Fng closed this Dec 27, 2025
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