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

Issues with Virga #5

Open
section83 opened this issue Aug 29, 2017 · 6 comments
Open

Issues with Virga #5

section83 opened this issue Aug 29, 2017 · 6 comments

Comments

@section83
Copy link

Hello,

I have been looking for macOS GUI's to download videos. Yours looks interesting. However, I've had some issues.

First is that the pbs command might need the "-update" option to work properly. The man page doesn't mention it but, I've found I get an error without it.

Second is that when I start Virga, I get a dialog asking me for the location of Google Chrome. I suspect that's because the code has this:

tell application "Google Chrome"

I don't understand AS well but, I think that when the script is run it checks the code, sees the "Tell" and hoping to avoid a future error, looks for Chrome. I've found the solution is to put "Google Chrome" into a variable (eg. google_chrome_app) and change that line to:

tell application google_chrome_app

Works for me anyway.

Third is that after cancelling that dialog, nothing happens. It suggests there is an AS error. When I try to run the Virga script in Script Editor I get this error "Expected end of line but found property". The error is on "tab" in this line:

tell application frontApp to set theURL to URL of active tab of front window

Hope this helps

Cheers.

@section83 section83 changed the title pbs command might need additions option Issues with Virga Aug 29, 2017
@kopurando
Copy link
Owner

thanks for notes, I will have a look,
but what is pbs? I may be missing something

@section83
Copy link
Author

Sorry, I was confusing Virga with another GUI called PPC Media Centre. Humble apologies.

By the way the pbs command updates the Service menu (and other things related). Ensures the new service will appear in the Services menu straight away rather than waiting for logoff etc. I've read via Google that it comes from the old NextStep in which it did more.

Cheers.

@kopurando
Copy link
Owner

btw, what browsers do you have installed?

@kopurando
Copy link
Owner

i think i fixed it, see the most recent commit. Will upload release bundles tomorrow.

@section83
Copy link
Author

I have Safari and Firefox installed.

Downloaded bundle just now [timed 9 hours ago]. Has same issues. See screen captures.

as seeking chrome
virga as error

I think the error is caused by the tab element of the code. I believe "tab" is a text constant in AS and so isn't treated as a property. Is there a way of turning reserved words into literals ?

Cheers.

@kopurando
Copy link
Owner

I see,
Well, virga works with Safari/Chrome (including betas) and requires youtube to be open in any of these browsers (but not Firefox). If no Safari / Chrome is running, then virga has nothing to do basically...

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

2 participants