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

Δεν δουλεύει σε mac #13

Closed
drpaneas opened this issue Sep 11, 2020 · 7 comments · Fixed by #43
Closed

Δεν δουλεύει σε mac #13

drpaneas opened this issue Sep 11, 2020 · 7 comments · Fixed by #43
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@drpaneas
Copy link

Διαλέξτε Σταθμό (Q/q για έξοδο): 1
+ [[ 1 = \q ]]
+ [[ 1 = \Q ]]
+ '[' 1 -gt 0 ']'
+ '[' 1 -le 3 ']'
./shelldio.sh: line 64: "1" : syntax error: operand expected (error token is ""1" ")
++ tail -n1
++ cut -d , -f1
+ stathmos_name=
./shelldio.sh: line 65: "1" : syntax error: operand expected (error token is ""1" ")

Επίσης αν η επιλογή σταθμού αποτύχει, πρέπει να κρασάρει και όχι να "νομίζει" ότι παίζει τον σταθμό.

@cerebrux cerebrux added the bug Something isn't working label Sep 11, 2020
@JohnGavr
Copy link

@drpaneas την λίστα των σταθμών στην εμφάνισε κανονικά;

@cerebrux cerebrux pinned this issue Sep 11, 2020
@drpaneas
Copy link
Author

@drpaneas την λίστα των σταθμών στην εμφάνισε κανονικά;

Ναι

@JohnGavr
Copy link

Έχω δημιουργήσει ένα branch features και ο τρόπος που γίνεται η αναπαραγωγή είναι λίγο διαφορετική.
Όταν βρεις χρόνο και μπορείς, δοκίμασε το και ενημέρωσε μας!

@drpaneas
Copy link
Author

Παλι δεν παίζει

line 149: "2" : syntax error: operand expected (error token is ""2" ")

Φαντάζομαι ότι οφείλεται σε ένα από τα παρακάτω:

  1. Δεν χρησιμοποιώ bash αλλά zshell
  2. Το Mac έχει ελαφρώς διαφορετική υλοποιήση των κλασσικών εντολών (βλ. sed) για παράδειγμα.

Γράψε ένα τεστάκι:

Δεδομένου ότι ο χρήστης έχει κανει git-clone και διαθέτει το mpv
Οταν τρέχει το script και επιλέγει τον 2ο σταθμό
Τότε η εντολή που πρέπει να τρέξει είναι η "τάδε"

@JohnGavr
Copy link

Ευχαριστώ για την απάντηση, αρχικά να ενημερώσω ότι και εγώ χρησιμοποιώ zshell και δεν αντιμετωπίζω κάποιο πρόβλημα. Λόγω του ότι δεν έχω στην κατοχή μου κάποιο mac δεν μπορώ να βοηθήσω περισσότερο.

@JohnGavr JohnGavr added the help wanted Extra attention is needed label Sep 17, 2020
@cerebrux cerebrux unpinned this issue Sep 17, 2020
drpaneas added a commit to drpaneas/shelldio that referenced this issue Sep 18, 2020
@drpaneas
Copy link
Author

@JohnGavr

Βαλε αυτό 7f390ac το patch και θα παίξει.

cerebrux added a commit that referenced this issue Sep 20, 2020
@cerebrux cerebrux linked a pull request Sep 20, 2020 that will close this issue
@cerebrux
Copy link

@drpaneas μπορεις να δεις λίγο το τελευταίο commit 785e638 (ή το pull request #43 )

Αν σου δουλεύει να πάει για merge

cerebrux added a commit that referenced this issue Sep 20, 2020
[REFACTOR] for 2.0.1
a496593 - [FIX] binaries check #34 (Salih Emin)
76a5714 - [FIX][DOC] remove from loop #35 (Salih Emin)
a5c3c10 - [FIX] exit code should be 1 #37 (Salih Emin)
287c714 - [REFACTOR] sort #39 (Salih Emin)
d5bf041 - [DOC] -y not -υ #44 (Salih Emin)
7782278 - [DOC]ορθογραφικό #40 (Salih Emin)
5f5cbf7 - [REFACTOR] fix #38 (Salih Emin)
785e638 - [FIX] RC για το macOS #13 (Salih Emin)
79005e3 - Merge pull request #33 from CerebruxCode/stable (Salih Emin)
a239151 - Merge branch 'stable' into develop (Salih Emin)
cerebrux added a commit that referenced this issue Sep 20, 2020
5480588 - Merge pull request #47 from zoispag/feat/add-station-sokfm (Salih Emin)
137011d - Drop ';listen.pls' from station url (Zois Pagoulatos)
502ad55 - Merge pull request #45 from zoispag/docs/mac-support (Salih Emin)
9d30e94 - [FEATURE] Προσθήκη σταθμού: Σοκ FM (Xαλκίδα) (Zois Pagoulatos)
277ead6 - Προσθήκη εντολής απεγκατάστασης για macOS (Zois Pagoulatos)
c3ba6d4 - Minor update (Zois Pagoulatos)
a496593 - [FIX] binaries check #34 (Salih Emin)
513ab47 - Αλλαγή MacOS σε macOS. Προσθήκη εντολής για symlink σε mac. (Zois Pagoulatos)
2a7fbd2 - [DOC] Προσθήκη οδηγιών για MacOS (Zois Pagoulatos)
76a5714 - [FIX][DOC] remove from loop #35 (Salih Emin)
a5c3c10 - [FIX] exit code should be 1 #37 (Salih Emin)
287c714 - [REFACTOR] sort #39 (Salih Emin)
d5bf041 - [DOC] -y not -υ #44 (Salih Emin)
7782278 - [DOC]ορθογραφικό #40 (Salih Emin)
5f5cbf7 - [REFACTOR] fix #38 (Salih Emin)
785e638 - [FIX] RC για το macOS #13 (Salih Emin)
79005e3 - Merge pull request #33 from CerebruxCode/stable (Salih Emin)
a239151 - Merge branch 'stable' into develop (Salih Emin)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
3 participants