Coder Social home page Coder Social logo

Comments (8)

setime avatar setime commented on July 26, 2024

Ugly workaround:

From ae9775c2ae929f5d7546d2dcbe24dd8cae878080 Mon Sep 17 00:00:00 2001
From: timeframe <[email protected]>
Date: Sat, 28 Oct 2023 10:15:06 +0200
Subject: [PATCH] Quick workaround to get the GUI working on Ubuntu 22.04

---
 src/common/pm3process.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/common/pm3process.cpp b/src/common/pm3process.cpp
index 76c13b8..9443421 100644
--- a/src/common/pm3process.cpp
+++ b/src/common/pm3process.cpp
@@ -32,6 +32,7 @@ void PM3Process::connectPM3(const QString& path, const QStringList args)
     start(path, args, QProcess::Unbuffered | QProcess::ReadWrite);
     if(waitForStarted(10000))
     {
+        QThread::msleep(5000);
         waitForReadyRead(10000);
         setRequiringOutput(false);
         result = *requiredOutput;
-- 
2.34.1

from proxmark3gui.

wh201906 avatar wh201906 commented on July 26, 2024

Preliminary analysis of the issue:
Using wayland promts in an additional message from the client. This is not expected from the GUI and breaks the connection. The issue is in the function PM3Process::connectPM3. Setting a breakpoint in the function is a work around, because then reading the output from the command line is delayed and the string os: is available in the result variable.

Proposed fix:
If the error message is in the result variable wait long until the expected string is available in the result variable.

Thank you for providing me these!

from proxmark3gui.

wh201906 avatar wh201906 commented on July 26, 2024

Would you like to make a PR for your workaround? I might implement the full fix you mentioned later.

from proxmark3gui.

setime avatar setime commented on July 26, 2024

Sure I can make a PR for this workaround. It delays the connection time, but it works.

from proxmark3gui.

wh201906 avatar wh201906 commented on July 26, 2024

That helps. Thanks!

from proxmark3gui.

setime avatar setime commented on July 26, 2024

PR is created to the dev branch. Just tell me if you want a PR to the master.

from proxmark3gui.

wh201906 avatar wh201906 commented on July 26, 2024

I've merged that. Thanks!

from proxmark3gui.

setime avatar setime commented on July 26, 2024

FYI: For anyone who wonders, the issue arises with the https://github.com/Proxmark/proxmark3 repo.

from proxmark3gui.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.