Skip to content

Fix: Resolve build failures and missing dependencies#7

Closed
stephanchenette wants to merge 1 commit intopedramamini:mainfrom
stephanchenette:fix-improve-macos-build
Closed

Fix: Resolve build failures and missing dependencies#7
stephanchenette wants to merge 1 commit intopedramamini:mainfrom
stephanchenette:fix-improve-macos-build

Conversation

@stephanchenette
Copy link
Copy Markdown

This commit resolves two issues that prevent the project from running on a fresh installation, particularly on macOS.

  • Switches the SQLCipher library from 'pysqlcipher3' to 'sqlcipher3' to fix compilation errors and allow the project to build successfully.

  • Adds 'tabulate' as an explicit dependency in setup.py, as it was a missing dependency for the activity_cli.py script.

This commit resolves two issues that prevent the project from running
on a fresh installation, particularly on macOS.

- Switches the SQLCipher library from 'pysqlcipher3' to 'sqlcipher3'
  to fix compilation errors and allow the project to build successfully.

- Adds 'tabulate' as an explicit dependency in setup.py, as
  it was a missing dependency for the activity_cli.py script.
@pedramamini
Copy link
Copy Markdown
Owner

@claude review this PR

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jul 9, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@pedramamini
Copy link
Copy Markdown
Owner

@claude are you there now?

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jul 9, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@pedramamini
Copy link
Copy Markdown
Owner

oh @claude why can't i get you back online?

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jul 9, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@pedramamini
Copy link
Copy Markdown
Owner

alright @claude let's try this again

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jul 9, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@pedramamini
Copy link
Copy Markdown
Owner

ok one more time @claude

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jul 9, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@pedramamini
Copy link
Copy Markdown
Owner

ok this time we should be working... @claude review this PR!

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jul 9, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@pedramamini
Copy link
Copy Markdown
Owner

@claude i've now changed everything from scratch. please work... review this PR

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jul 9, 2025

Claude encountered an error —— View job

Failed with exit code 128

I'll analyze this and get back to you.

@pedramamini
Copy link
Copy Markdown
Owner

@stephanchenette please resubmit a fresh PR, somehow the original branch seems to be missing.

@pedramamini pedramamini closed this Jul 9, 2025
@stephanchenette stephanchenette deleted the fix-improve-macos-build branch July 10, 2025 05:05
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