Skip to content

Conversation

@dustinmcafee
Copy link
Collaborator

  • Extract all encoding implementations to separate rfb-encodings crate
  • Maintain full backwards compatibility through re-exports
  • Use published rfb-encodings 0.1.2 from crates.io
  • Benefits: reusable encodings, cleaner architecture, better discoverability

- Extract all encoding implementations to separate rfb-encodings crate
- Maintain full backwards compatibility through re-exports
- Use published rfb-encodings 0.1.2 from crates.io
- Benefits: reusable encodings, cleaner architecture, better discoverability
Updated rfb-encodings dependency from 0.1 to 0.1.3 to fix critical build
failure when using turbojpeg feature without debug-logging feature.
- Added references to rfb-encodings throughout README, TECHNICAL, and CONTRIBUTING
- Updated architecture diagrams to show separation between rustvncserver and rfb-encodings
- Clarified that rfb-encodings provides encoding implementations for servers
- Updated version examples to 2.0
- Bumped version to 2.0.0-beta.4
Remove duplicate code and improve maintainability:
- Updated rfb-encodings dependency from 0.1.3 to 0.1.5
- Removed duplicate encoding constants (now imported from rfb-encodings)
- Deleted duplicate src/jpeg/ module (now using TurboJPEG from rfb-encodings)
- Code reduction: ~220 lines of duplicate code eliminated

Add comprehensive TurboJPEG documentation and licensing information:
- Added TurboJPEG installation instructions for all platforms
- Updated NOTICE file with complete libjpeg-turbo attribution
- Documented BSD-3-Clause, IJG, and zlib license components
- Clarified that libjpeg-turbo is NOT distributed with this server
@dustinmcafee dustinmcafee merged commit 19b079a into main Oct 27, 2025
9 checks passed
dustinmcafee added a commit that referenced this pull request Oct 27, 2025
feat: v2.0.0-beta.1 - extract encodings to rfb-encodings crate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants