why after install/uninstall application can not terminate other application

Fixed

Description

I am trying to make a qt application liked app store, in this application I can install/uninstall/launch/terminate application.When I operated this application as below sequence, it can't terminate application at step2.

  1. install/uninstall a application A

  2. terminate another application B

  3. uninstall application B

My code

config.xml

Is there existing fault in my code? how can I using terminate and install/uninstall at the same time?

Additional remarks:

  • After step1,if I transfer screen to launcher, then back to app store application, I can terminate application B.

  • you can reappear this issue by afb-client-demo
    afb-client-demo --human 'ws://localhost:1042/api?token=HELLO'
    afm-main uninstall dashboard@0.1
    afm-main terminate hvac@0.1

Environment

None

Activity

Show:

zhiqiang wang 
November 7, 2018 at 1:41 AM

I'm glad I could be of some assistance.I have tested this patch,it's OK.

jose bollo 
November 5, 2018 at 5:44 PM

fix in review

jose bollo 
November 5, 2018 at 8:53 AM

Shame on me! I let kill(0) meaning that it will be a no-op when the app is not started but I just forgot that it means kill-me! I'll fix it now. Thank again and again...

zhiqiang wang 
November 1, 2018 at 6:33 AM
(edited)

I tested the latest version in master branch, and found one issue and one doubt.

  • When dashboard isn't started, used "afm-util teminate dashboard@0.1" in command line, then got below logs:

I thought afm-main was restarted.

IMHO, maybe the operation "terminate application that doesn't start" is a mistake, but I
thought afm-main do nothing under this scenario is better than restart.

  • When dashboard application was not started, I use "afm-util state dashboard@0.1" to get application's state,then I got "ERROR: failed No such file or directory", in previous version I got "null". is this an improvement in new version?(when terminate failed also got this log)

remarks

I use "rpm -ivh *.rpm --force" to install four rpm files.

  • af-main-dbg-master+git0+c152727a73-r0.aarch64.rpm

  • af-main-dev-master+git0+c152727a73-r0.aarch64.rpm

  • af-main-master+git0+c152727a73-r0.aarch64.rpm

  • af-main-tools-master+git0+c152727a73-r0.aarch64.rpm

jose bollo 
October 28, 2018 at 6:05 PM

I pushed the fix. Thank you for the report.

Details

Assignee

Reporter

Labels

Contract ID

Priority

Created October 25, 2018 at 7:41 AM
Updated December 7, 2018 at 11:36 PM
Resolved November 30, 2018 at 8:39 AM