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

script name can be followed by optional arguments #12

Closed
wants to merge 1 commit into from
Closed

script name can be followed by optional arguments #12

wants to merge 1 commit into from

Conversation

polakovic
Copy link
Contributor

Script definition now can contain also optional arguments (e.g. for using single script for multiple operation).

@polakovic
Copy link
Contributor Author

Hm, what are the conflicts? I don't see any other pending pull request for these two files...

@sholmbo
Copy link
Contributor

sholmbo commented May 30, 2016

Your branch is behind indilib:master which has changes to telescope_script.cpp that are not in yours. So you have to merge indilib:master with yours first.

@polakovic
Copy link
Contributor Author

Sorry, but how can I do it?

@knro
Copy link
Contributor

knro commented May 30, 2016

Just do a git pull, and git merge master.

@polakovic
Copy link
Contributor Author

I tried already, it doesn't do anything:

retina:indi polakovic$ git pull
Already up-to-date.
retina:indi polakovic$ git merge master
Already up-to-date.
retina:indi polakovic$

@sholmbo
Copy link
Contributor

sholmbo commented May 30, 2016

You have to merge the indilib:master as it is the one that should pull. If you haven't added it as a remote, try something like

$ git remote add indilib git@github.com:indilib/indi.git
$ git pull indilib master

Then you'll get the merge conflict I think.

@polakovic
Copy link
Contributor Author

fatal: Could not read from remote repository.
Please make sure you have the correct access rights and the repository exists.

Both for git@github.com and polakovic@github.com...

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

Successfully merging this pull request may close these issues.

3 participants