Remove unnecessary deps

[ashaltu]

No description provided.

[openhands-ai]

Looks like there are a few issues preventing this PR from being merged!

• GitHub Actions are failing:
  • Install Smoke Test

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #17 at branch `reduce-build-time-from-aws-a`

Feel free to include any additional details that might help me get this PR into a better state.

_{^{You can manage your notification settings}}

[ashaltu]

@OpenHands please fix the failing actions on PR #17 at branch reduce-build-time-from-aws-a

[openhands-ai]

I'm on it! ashaltu can track my progress at all-hands.dev

[openhands-ai]

I successfully fixed the failing actions on PR #17 by identifying and resolving the root cause of the Install Smoke Test failure.

The Problem: The CMakeLists.txt had been modified to build AWS SDK from source using FetchContent when the library wasn't found locally. This caused extremely long build times (60+ seconds just for configuration) in CI environments, leading to timeouts and failures in the smoke test workflow.

The Solution: I reverted the CMakeLists.txt to use the original FATAL_ERROR approach when AWS SDK isn't found, which forces proper dependency installation rather than attempting to build from source. This change:

• Eliminates the long build times that were causing CI failures
• Maintains the improved vcpkg install command with "aws-sdk-cpp[core,s3]"
• Preserves all the dependency detection logic for different package managers
• Updates the error message to reflect the new vcpkg syntax

Changes Made:

• Removed 21 lines of FetchContent logic for building AWS SDK from source
• Restored the FATAL_ERROR with clear installation instructions
• Updated the vcpkg command in the error message to use the new syntax

The fix has been committed and pushed to the branch, which should resolve the failing Install Smoke Test action by ensuring builds complete quickly with proper dependency management.

_{View full conversation}