kittoku / Open-SSTP-Client

Open SSTP Client for Android
MIT License
402 stars 104 forks source link

Hello, I just tried to make this feature by myself. But I got caught in a cyclic dependency problem. #104

Closed Hoseinnikvarz closed 1 year ago

Hoseinnikvarz commented 1 year ago

Hello, I just tried to make this feature by myself. But I got caught in a cyclic dependency problem.

Following is a sequence of my approach: (1) Make 'Run Script' menu and command in 'org.jkiss.dbeaver.ui.navigator' (2) Write handler setting in plugin.xml to connect the command with 'SQLEditorHandlerExecute' class in 'org.jkiss.dbeaver.ui.editor.sql' (3) Add 'org.jkiss.dbeaver.ui.editor.sql' in Required-Bundle of MANIFEST (4) Because 'org.jkiss.dbeaver.ui.editor.sql' already has dependency on 'org.jkiss.dbeaver.ui.navigator', (3) raises cyclic dependency and compile fails

I tried to find a way to execute SQL without 'SQLEditorHandlerExecute' class, but I couldn't make it. Could anyone have better approach to make this feature? Any advice would be appreciated!

Originally posted by @CodyBuilder-dev in https://github.com/dbeaver/dbeaver/issues/4738#issuecomment-803351501