maemo.org - Talk
Page 1 of 4 1 23 Last »
Show 40 post(s) from this thread on one page

maemo.org - Talk (https://talk.maemo.org/index.php)
-   Applications (https://talk.maemo.org/forumdisplay.php?f=41)
-   -   [Announce][Fremantle] Ten Minute Core Workout (https://talk.maemo.org/showthread.php?t=87451)

sixwheeledbeast 2012-10-18 20:48
[Announce][Fremantle] Ten Minute Core Workout
 
4 Attachment(s)
I'd like to announce Ten Minute Core Workout.

All information on the wiki page.

Tiran 2012-10-19 18:36
Re: [Announce/WIP] Ten Minute Core Workout
 
Please, place here screenshots of program Ten Minute Core Workout!

sixwheeledbeast 2012-10-19 19:11
Re: [Announce/WIP] Ten Minute Core Workout
 
Done.
It's not the most exciting screenshot ever.

Can anybody help with my buttons, would there be a reason why these slots would not work?

Code:
void MainWindow::on_pushButtonStart_clicked()
{
    QProcess process;
    process.start("sh /opt/tenminutecore/script/tenminutecore.sh");
}

void MainWindow::on_pushButtonStop_clicked()
{
    QProcess process;
    process.start("sh /opt/tenminutecore/script/kill.sh");
    process.waitForFinished();
}

Copernicus 2012-10-19 19:40
Re: [Announce/WIP] Ten Minute Core Workout
 
Hmm, not sure why that wouldn't be working; it looks exactly like what the Qt documentation says for QProcess.

Maybe there's some sort of permissions problem?

There could also be some issue with how the argument is being passed to the shell, I suppose. You might try creating a shell script instead, i.e.:

Code:
#!/bin/sh

/opt/tenminutecore/script/tenminutecore.sh
(Assuming that tenminutecore.sh isn't already an executable shell script itself.) Then make it executable, and just use the script name instead:

Code:
process.start("yourscriptname.sh");
If all else fails, you can always fall back to using the good old fork/exec mechanism:

Code:
  pid_t pid = fork();

  if (pid == 0)
  {
    // We're inside the child process
    execl(
      "/bin/sh",
      "/bin/sh",
      "/opt/tenminutecore/script/tenminutecore.sh",
      NULL);
  }

  // If we reach this point, we are inside the parent process.
  // So, we'll wait for the child process to complete:
  int *stat_loc = NULL;
  waitpid(pid, stat_loc, 0);
Not portable to non-linux systems the way that QProcess is, but I can at least say that this works fine in Pierogi. :)

EDIT: There is apparently an "error()" signal for the QProcess object that gets sent if the call to "start()" fails. Maybe connecting that to a slot to see what it says will provide a bit more info as to what is going on...

sixwheeledbeast 2012-10-19 20:09
Re: [Announce/WIP] Ten Minute Core Workout
 
@Copernicus Thanks for the ideas.

Seems changing to this did the trick (pinched from Theme Customizer)
No idea why the first version didn't work.

Code:
void MainWindow::on_pushButtonStart_clicked()
{
    QProcess *process;
            process = new QProcess( this );
            process->start("sh opt/tenminutecore/script/tenminutecore.sh");
}

void MainWindow::on_pushButtonStop_clicked()
{
    QProcess *process;
            process = new QProcess( this );
            process->start("sh opt/tenminutecore/script/kill.sh");
            process->waitForFinished();
}
Just require some testers now.

Finally what's the best way to go about implementing portrait support?
Ideally I just require the stop button to move below the start in portrait.

Copernicus 2012-10-19 20:28
Re: [Announce/WIP] Ten Minute Core Workout
 
Quote:
Originally Posted by sixwheeledbeast (Post 1282695)
Seems changing to this did the trick (pinched from Theme Customizer)
No idea why the first version didn't work.
It might be that QProcess needed to have the parent object specified. (Or maybe the fact that the Qt guys are in love with the model-view-controller paradigm, and therefore want everything allocated on the heap rather than the stack. :) I'm not a huge fan of MVC architecture...) Good to see that it's working, though!

I can't help much with properly supporting portrait mode, but you probably knew that already. :)

marxian 2012-10-19 21:16
Re: [Announce] Ten Minute Core Workout
 
Your old approach didn't work because the QProcess object fell out of scope. Your second approach seems to result in a memory leak, since you are creating new QProcess objects when the buttons are pressed, without deleting them until the MainWindow object is deleted (since that is the parent). The best solution will be to make it a private member of MainWindow:

Code:
private:
    QProcess *m_process;
Instantiate it in the MainWindow constructor (and connect to its signals for error checking if you want) and you can use the same QProcess object to launch the scripts.

For portrait support, you can connect to the resized() signal of QDesktopWidget and adjust the geometry of your widgets as you require.

Code:
#include <QApplication>
#include <QDesktopWidget>

...

connect(QApplication::desktop(), SIGNAL(resized(int)), this, SLOT(onOrientationChanged()));

...

void MainWindow::onOrientationChanged() {
  QRect screen = QApplication::desktop()->screenGeometry();

    if (screen.width() > screen.height()) {
        setLandscapeLayout();
    }
    else {
        setPortraitLayout();
    }
}

void MainWindow::setLandscapeLayout() {
  do stuff
}

void MainWindow::setPortraitLayout() {
  do stuff
}

sifo 2012-10-19 21:51
Re: [Announce] Ten Minute Core Workout
 
What about moving this thread to Development section ?

Copernicus 2012-10-19 23:33
Re: [Announce] Ten Minute Core Workout
 
Quote:
Originally Posted by marxian (Post 1282728)
Your old approach didn't work because the QProcess object fell out of scope.
Ah! So a "StartDetached()" call would work in that case, allowing the script to continue after the QProcess object is destroyed.

Actually, this also means you don't really need the "kill.sh" script to stop the process (unless kill.sh does something special). A call to "terminate()" (or just destroying the object) should do the trick...

EDIT: As Marxian notes below, you can't call "terminate()" on a destroyed QProcess object. :) (In short, you'd need to have a QProcess object that lives beyond the start_clicked() method, call "start()" (not "startDetached()") on it in that method, and call "terminate()" in the stop_clicked() method...)

marxian 2012-10-19 23:53
Re: [Announce] Ten Minute Core Workout
 
Quote:
Originally Posted by Copernicus (Post 1282774)
Ah! So a "StartDetached()" call would work in that case, allowing the script to continue after the QProcess object is destroyed.

Actually, this also means you don't really need the "kill.sh" script to stop the process (unless kill.sh does something special). A call to "terminate()" (or just destroying the object) should do the trick...
startDetached() will work, yes, because it's a static method. The disadvantage is that you don't have access to the QProcess object after the method is called. Another alternative is to call waitForFinished() after calling start(), which enables synchronous (blocking) use of QProcess. That way you can use a QProcess object with local scope, because there is no further event processing until the start() method returns. This approach has the disadvantage of blocking the main thread until the program called by QProcess exits. Synchronous use of QProcess is really only useful for cases where there is no event loop. I can't see any reason to use it where you have a QApplication instance.


All times are GMT. The time now is 13:10.
Page 1 of 4 1 23 Last »
Show 40 post(s) from this thread on one page

vBulletin® Version 3.8.8