screenpipe icon indicating copy to clipboard operation
screenpipe copied to clipboard

[bounty] prevent double screenpipe backend running at once

Open louis030195 opened this issue 1 year ago • 2 comments

/bounty 100

definition of done:

  • app always avoid having two screenpipe backend running at once

louis030195 avatar Nov 28 '24 16:11 louis030195

~~## 💎 $100 bounty • Screenpi.pe~~

~~### Steps to solve:~~ ~~1. Start working: Comment /attempt #795 with your implementation plan~~ ~~2. Submit work: Create a pull request including /claim #795 in the PR body to claim the bounty~~ ~~3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts~~

~~Thank you for contributing to mediar-ai/screenpipe!~~

~~Add a bountyShare on socials~~

Attempt Started (GMT+0) Solution
🟢 @ologbonowiwi Dec 3, 2024, 2:39:26 PM #900

algora-pbc[bot] avatar Nov 28 '24 16:11 algora-pbc[bot]

/attempt #795

Algora profile Completed bounties Tech Active attempts Options
@ologbonowiwi    5 mediar-ai bounties
+ 53 bounties from 7 projects
Rust, TypeScript,
Shell & more
Cancel attempt

wesleymatosdev avatar Dec 03 '24 14:12 wesleymatosdev

adding a bit more context:

keep seeing windows users having issues with double screenpipe running or stuff like this

i rarely have this issue on mac

the auto destruct flag in CLI make the CLI kill itself when the app is gone - although windows is such hard to predict OS haha

also other context:

i sometimes have double screenpipe running intentionally because i run a production screenpipe and then i run another dev screenpipe on a different port and data dir - i should still be able to do that

louis030195 avatar Dec 03 '24 16:12 louis030195

used a check to see wether the port is available to be used or not, in this way you'll still be able to run multiple screenpipe instance @louis030195.

let me know what you think

wesleymatosdev avatar Dec 03 '24 20:12 wesleymatosdev

let's move the check if the port is taken earlier, so we immediately return it to the user before kicking of any other CLI processes, this way we can get a cleaner error output, which will help new users

m13v avatar Dec 09 '24 03:12 m13v

@m13v on #836 the check is already on the beginning of the method. still waiting for approval though

wesleymatosdev avatar Dec 09 '24 03:12 wesleymatosdev

or you mean add even before that function? If so I'll make the change.

wesleymatosdev avatar Dec 09 '24 03:12 wesleymatosdev

just trying to get this error log to be the first one when you start CLI so users don't get bogged down by details of the logs

On Sun, Dec 8, 2024 at 7:52 PM ologbonowiwi @.***> wrote:

or you mean add even before that function? If so I'll make the change.

— Reply to this email directly, view it on GitHub https://github.com/mediar-ai/screenpipe/issues/795#issuecomment-2526803574, or unsubscribe https://github.com/notifications/unsubscribe-auth/AY62CDGCRG63TVCUBRWD5CL2EUHZNAVCNFSM6AAAAABSVP2CLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRWHAYDGNJXGQ . You are receiving this because you were mentioned.Message ID: @.***>

m13v avatar Dec 09 '24 19:12 m13v

💡 @ologbonowiwi submitted a pull request that claims the bounty. You can visit your bounty board to reward.

algora-pbc[bot] avatar Dec 10 '24 14:12 algora-pbc[bot]

@ologbonowiwi: You've been awarded a $100 bounty by mediar-ai! We'll notify you once it is processed.

algora-pbc[bot] avatar Dec 11 '24 16:12 algora-pbc[bot]

🎉🎈 @ologbonowiwi has been awarded $100! 🎈🎊

algora-pbc[bot] avatar Dec 12 '24 20:12 algora-pbc[bot]