Applications
/
ublexec
9
0
Fork 2
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity
Labels Milestones
New Issue

Возможные варианты эмуляторов термина вынести в массив, с зависящими входящими аргументами #15

Closed
opened 3 years ago by asmeron · 11 comments
asmeron commented 3 years ago
Owner

Перечень возможных терминалов вынести в массив с аргументами, который описать в заголовке !

  • xfce4-terminal -x
  • konsole -e
  • xterm -e

Originally posted by @asmeron in #13

Перечень возможных терминалов вынести в массив с аргументами, который описать в заголовке ! - xfce4-terminal -x - konsole -e - xterm -e _Originally posted by @asmeron in https://gitea.ublinux.ru/Applications/ublexec/pulls/13#issuecomment-1536_
asmeron added the улучшение label 3 years ago
Igor1 was assigned by asmeron 3 years ago
Igor1 commented 3 years ago

Сделал.
2efaff2e39

Сделал. https://gitea.ublinux.ru/Igor1/ublexec/commit/2efaff2e39f947db51b1638cf83f3fb6a7ecf56d
asmeron commented 3 years ago
Poster
Owner

Зачем два одномерных массива ??
Сделать ОДИН массив !!!

Зачем два одномерных массива ?? Сделать ОДИН массив !!!
Igor1 commented 3 years ago

Название и команда. Нужно ДВА массива !!!
Конкантенацией не решишь, разные параметры.
А делать срез, это костыль.
{"konsole", "xfce4-terminal", "xterm"};
{"konsole -e ", "xfce4-terminal -x ", "xterm -e "};

Название и команда. Нужно ДВА массива !!! Конкантенацией не решишь, разные параметры. А делать срез, это костыль. {"konsole", "xfce4-terminal", "xterm"}; {"konsole -e ", "xfce4-terminal -x ", "xterm -e "};
Igor1 commented 3 years ago

Немного переделал.
fd62bece01

Немного переделал. https://gitea.ublinux.ru/Igor1/ublexec/commit/fd62bece01d63ed4d1a89653bc5662c7000a3039
asmeron commented 3 years ago
Poster
Owner

Сделай тогда один двухмерный массив, твое решение не устраивает !

Сделай тогда один двухмерный массив, твое решение не устраивает !
Igor1 commented 3 years ago

Ок, но будет через срез.

Ок, но будет через срез.
Igor1 commented 3 years ago

Сделал.
728d996b67

Сделал. https://gitea.ublinux.ru/Igor1/ublexec/commit/728d996b67ecd8c75eb1c367169a73f09bd4b2c8
asmeron commented 3 years ago
Poster
Owner

Ничего не сделал, терминалов будет больше, и у каждого будут свои аргументы, сейчас не известные. Ты в коде жёстко прописываешь аргументы -x -e.
Код не должен быть жёстко привязан к конкретным аргументам конкретных терминалов.

Этого не должно быть !

str_remove(name_terminal, " -e ");
str_remove(name_terminal, " -x ");

Что тебе из этого не понятно ???

Ничего не сделал, терминалов будет больше, и у каждого будут свои аргументы, сейчас не известные. Ты в коде жёстко прописываешь аргументы -x -e. Код не должен быть жёстко привязан к конкретным аргументам конкретных терминалов. Этого не должно быть ! ```c str_remove(name_terminal, " -e "); str_remove(name_terminal, " -x "); ``` Что тебе из этого не понятно ???
Igor1 commented 3 years ago

Так ?
8ab1e8a7cf

Так ? https://gitea.ublinux.ru/Igor1/ublexec/commit/8ab1e8a7cf80535d26422f6b9f2d49e17eb0ca59
asmeron commented 3 years ago
Poster
Owner

Пойдёт, самое главное, что-бы понятно было для чего так нужно реализовать !

Пойдёт, самое главное, что-бы понятно было для чего так нужно реализовать !
user-166 commented 3 years ago
Collaborator

Проверил. Закрываю.

Проверил. Закрываю.
user-166 closed this issue 3 years ago
asmeron referenced this issue from a commit 3 years ago
Исправил замечание #15
user-166 added this to the Версия 1.x milestone 3 years ago
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
master
Labels
Apply labels
Clear labels
вопрос
Требуется дополнительная информация дизайн
Проблема с дизайном дубликат
Эта проблема или запрос на включение уже существует неисправимо
Это не будет исправлено неисправно
Что-то неисправно ошибка
Что-то не работает помощь
Нужна помощь улучшение
Новая возможность
No Label вопрос дизайн дубликат неисправимо неисправно ошибка помощь улучшение
Milestone
Set milestone
Clear milestone
No items
No Milestone
Версия 1.x
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Applications/ublexec#15
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(<nil>)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes