Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add support yad and qarma #661

Closed
ghost opened this issue Jul 1, 2016 · 11 comments
Closed

add support yad and qarma #661

ghost opened this issue Jul 1, 2016 · 11 comments

Comments

@ghost
Copy link

ghost commented Jul 1, 2016

since both are zenity compatible it should be very easy.

see also:

@austin987
Copy link
Contributor

Can you please test aff70bb

Note that it's in a branch, not in master

@ghost
Copy link
Author

ghost commented Jul 11, 2016

qarma-qt5 works for me, yad seems to use a slightly different syntax here

for example:

qarma|yad \
            --title 'a' \
            --text 'b' \
            --list \
            --radiolist \
            --column '' \
            --column '' \
            --column '' \
            --height 600 \
            --width 800 \
            --hide-column 2 \
            FALSE help       'c' \
            FALSE apps       'd' \
            FALSE benchmarks 'd' \
            FALSE games      'e' \
            TRUE  main       'f' \

apps selection qarma output:

apps

apps selection yad output:

TRUE|apps|d|

so to get the same output as qarma i would need something like:

echo "$?"|cut -d'|' -f2

or add those flags:

--print-column=2 --separator=''

@austin987
Copy link
Contributor

austin987 commented Jul 12, 2016

I did that 714bf88. Some stuff works, but it's still failing when e.g., changing settings.

@ghost
Copy link
Author

ghost commented Jul 12, 2016

if I add

--print-column=2 --separator=''  

to the command @line3004 yad works too(I havent checked all, just ddr=gdi)

@austin987
Copy link
Contributor

1da81f8 fixes settings, but dlls do not work.

@austin987
Copy link
Contributor

@sixsixfive could you please test when you get a chance?

@ghost
Copy link
Author

ghost commented Jul 23, 2016

sorry I'm currenly a bit busy but i tried this today, however im still unsure (the script is a bit messy ;),

i assume

LANG=C sh winetricks dlls 

calls line 3101 ?

if so even with zenity it will not display more than the first 3 columns so there is probably a bug somewhere @Line3251


PS: to detect the possible guis something like this would be much smaller:

winetricks_detect_gui() {
    WINETRICKS_MENU_HEIGHT=500
    WINETRICKS_MENU_WIDTH=1010
    _guis="kdialog zenity yad qarma"
        for _gui in $_guis; do
            type $_gui &>/dev/null && WINETRICKS_GUI=$_gui && printf "GUI will be $_gui\n" 
        done
        if [ "$_gui" = "kdialog" ];then
            printf "Zenity not found! Using kdialog as poor substitute.\n"
        fi
        if [ -z $WINETRICKS_GUI ];then
            cat <<EOF

No arguments given, so tried to start GUI, but zenity not found.
Please install zenity if you want a graphical interface, or
run with --help for more options."

EOF
            exit 1
        fi
}

@ghost
Copy link
Author

ghost commented Oct 2, 2016

I close this now since winetricks is now working fine with qarma ;)

@ghost ghost closed this as completed Oct 2, 2016
@austin987
Copy link
Contributor

Those changes aren't in master..

@ghost
Copy link
Author

ghost commented Oct 3, 2016

no but there was an update for qarma and it usually installs a symlink as zenity

@austin987
Copy link
Contributor

Even better, thanks for info.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant