master #11

Merged
asmeron merged 110 commits from Igor1/ubl-settings-bootloader:master into develop 3 years ago
Igor1 commented 3 years ago

Частично сделано сохранение. Параметры "Логин защиты", "Пароль защиты","Загрузка по умолчанию" не работают.

Частично сделано сохранение. Параметры "Логин защиты", "Пароль защиты","Загрузка по умолчанию" не работают.
Igor1 added 46 commits 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 2 commits 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 3 commits 3 years ago
Igor1 added 7 commits 3 years ago
Igor1 added 10 commits 3 years ago
Igor1 added 4 commits 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
asmeron requested changes 3 years ago
asmeron left a comment
Owner

Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h

Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h
Makefile Outdated
DEPENDS = /bin/cmake
PREFIX ?= /usr
PKGNAME = $(MAKEFILE_DIR)
#PKGNAME = ubl-settings-datetime
Owner

ubl-settings-datetime ??

ubl-settings-datetime ??
Igor1 commented 3 years ago
Poster

Исправил d54cae5d80

Исправил https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/d54cae5d80bf660add8a571a57bc6e20a92dc86c
Makefile Outdated
init:
@echo "Initialize ..."; \
sed -r "s/(version_application = ).*/\1\"$$(grep 'VERSION' ${FILE_VERSION} | cut -d" " -f2)\";/" -i source/ubl-settings-datetime.cc; \
Owner

ubl-settings-datetime ??

ubl-settings-datetime ??
Igor1 commented 3 years ago
Poster

Исправил d54cae5d80

Исправил https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/d54cae5d80bf660add8a571a57bc6e20a92dc86c
<description>Run "ubl-settings-bootloader" as root</description>
<description xml:lang="ru">Запус "Загрузка системы" с правами root</description>
<message>Authentication is required to run "ubl-settings-bootloader"</message>
<message xml:lang="ru">Требуется авторизация для запуска "Загрузка системы" с правами root</message>
Owner

"Загрузка системы" ??

"Загрузка системы" ??
Igor1 commented 3 years ago
Poster

Что не так ?
В тз "Загрузка системы"

Что не так ? В тз "Загрузка системы"
Igor1 commented 3 years ago
Poster

2d72e52ffd

Заменил на "ubl-settings-bootloader"

https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/2d72e52ffd981a026521eb1e685c4f93dc0ffbdb Заменил на "ubl-settings-bootloader"
string response = "";
string key = "";
string value = "";
cmd = "ubconfig --default --source " + str_flag_load + " get " + this->sections;
Owner

Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h

Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h
Igor1 commented 3 years ago
Poster

У меня все грузиться одной командой. У меня универсальное решение которое можно использовать с минимальными затратами времени, есть возможность масштабирования. Выносить все в #define когда команду можно генерировать исходя из параметров зачем ? Мы же не на 1 курсе. Из-за того что я использую словари у меня скорость выполнения поиска по словарю Min(O(1)) -> Max(O(n)). Когда я спросил у Вани какое у тебя О больше, он сказал я не знаю что это.

У меня все грузиться одной командой. У меня универсальное решение которое можно использовать с минимальными затратами времени, есть возможность масштабирования. Выносить все в #define когда команду можно генерировать исходя из параметров зачем ? Мы же не на 1 курсе. Из-за того что я использую словари у меня скорость выполнения поиска по словарю Min(O(1)) -> Max(O(n)). Когда я спросил у Вани какое у тебя О больше, он сказал я не знаю что это.
Owner

Игорь, консольные команды это не часть языка С++, это внешние вызовы, их удобнее в случае проблем или изменений править в одном месте.
Значения так-же подстановкой передаваемым преременным можно указывать, вот и получитаться, что ты генерируешь изменения которые предеаются в заранее сконструированный синтаксис вызова внешней команды.

Это никак не влияет на модульность или KISS. Это только влияет на стиль и удобочитаемость. Большой + что команды в случае изменения легко подправить, не погружаясь в чересчур лишнюю генерацию !

Либо предложи компромисс, между как у Ивана и как у тебя !

Игорь, консольные команды это не часть языка С++, это внешние вызовы, их удобнее в случае проблем или изменений править в одном месте. Значения так-же подстановкой передаваемым преременным можно указывать, вот и получитаться, что ты генерируешь изменения которые предеаются в заранее сконструированный синтаксис вызова внешней команды. Это никак не влияет на модульность или KISS. Это только влияет на стиль и удобочитаемость. Большой + что команды в случае изменения легко подправить, не погружаясь в чересчур лишнюю генерацию ! Либо предложи компромисс, между как у Ивана и как у тебя !
Igor1 commented 3 years ago
Poster

Я к тому что в одном проекте 5 команд в другом 10 и каждый раз править код ?

Я к тому что в одном проекте 5 команд в другом 10 и каждый раз править код ?
Igor1 commented 3 years ago
Poster

Есть вариант, ключи словаря они же параметры ubconfig вынести в отдельный заголовочный файл.

Есть вариант, ключи словаря они же параметры ubconfig вынести в отдельный заголовочный файл.
Owner

Почему не сделать, как делает Иван ?
Я не вижу ни одного минуса !

Почему не сделать, как делает Иван ? Я не вижу ни одного минуса !
Igor1 commented 3 years ago
Poster

У Ивана самописанные словари, зачем мне с++ изобретать велосипеды, когда они уже есть в С++ ?

У Ивана самописанные словари, зачем мне с++ изобретать велосипеды, когда они уже есть в С++ ?
Igor1 commented 3 years ago
Poster

Исправил.
65f13c4662

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/65f13c466237fbd1cf196a1f832fcd28cc87c34d
value = (*map_gui)[key];
if (this->check_save(str_flag_save, key)) {
if (value.length() != 0) {
cmd = "ubconfig --target " + str_flag_save + " set " + sections + " " +
Owner

Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h

Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h
Igor1 commented 3 years ago
Poster

Я стараюсь развивать модульность, универсальность , а не фигачить все в два файла. Пытаюсь следовать методике KISS не всегда получается, но я стараюсь

Я стараюсь развивать модульность, универсальность , а не фигачить все в два файла. Пытаюсь следовать методике KISS не всегда получается, но я стараюсь
Igor1 commented 3 years ago
Poster

Исправил.
65f13c4662

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/65f13c466237fbd1cf196a1f832fcd28cc87c34d
key + "=\"" + value + "\"";
}
else if (value.length() == 0) {
cmd = "ubconfig --target " + str_flag_save + " remove " + sections + " " + key;
Owner

Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h

Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h
Igor1 commented 3 years ago
Poster

Я стараюсь развивать модульность, универсальность , а не фигачить все в два файла. Пытаюсь следовать методике KISS не всегда получается, но я стараюсь

Я стараюсь развивать модульность, универсальность , а не фигачить все в два файла. Пытаюсь следовать методике KISS не всегда получается, но я стараюсь
Igor1 commented 3 years ago
Poster

Исправил.
65f13c4662

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/65f13c466237fbd1cf196a1f832fcd28cc87c34d
# SOME DESCRIPTIVE TITLE.
Owner

Реквизиты сделать правильные

Реквизиты сделать правильные
Igor1 commented 3 years ago
Poster

Исправил d54cae5d80

Исправил https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/d54cae5d80bf660add8a571a57bc6e20a92dc86c
# Russian translations for PACKAGE package.
Owner

Реквизиты сделать правильные

Реквизиты сделать правильные
Igor1 commented 3 years ago
Poster

Исправил d54cae5d80

Исправил https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/d54cae5d80bf660add8a571a57bc6e20a92dc86c
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
asmeron requested changes 3 years ago
treeViewIPT->set_model(list_store_IPT);
treeViewOTT->set_model(list_store_OTT);
vec_Option_kernel = this->read_file_and_view("cmdline-linux.csv", row_kernel, list_store_kernel);
Owner

Имена внешних файлов так-же в переменные !
Исправить везде по тексту !

Имена внешних файлов так-же в переменные ! Исправить везде по тексту !
Igor1 commented 3 years ago
Poster

Исправил.
7ba4533fc0

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/7ba4533fc0db6da2006bea16d76a6e023f4c9e5f
btnLoad->set_label(_("Load"));
this->set_title(_("ubl-settings-bootloader"));
// Todo: check
entrySecurityLogin->set_tooltip_text(gettext("User name to enter boot configuration editing mode"));
Owner

Нет единого стиля, если используешь для локализации функцию _ то и используй её, зачем то так то сяк gettext ?

Нет единого стиля, если используешь для локализации функцию _ то и используй её, зачем то так то сяк gettext ?
Igor1 commented 3 years ago
Poster

Заменил.
412495a222

Заменил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/412495a2226f11f75f99c5c452de8fe3616887dd
bool MainWindow::focus_out_txt_password(GdkEventFocus*) {
if (entryPasswordProtecc->get_text().length() == 0) {
info_status_app("boxInfoMessError");
imgInfo->set_from_icon_name("com.ublinux.ubl-settings-bootloader.warning", Gtk::ICON_SIZE_MENU);
Owner

Старайся все имена внешних файлов и прочего внешнего, выносить в переменные которые описывать в одном месте !
Везде по тексту исправь аналогичные ошибки.

Старайся все имена внешних файлов и прочего внешнего, выносить в переменные которые описывать в одном месте ! Везде по тексту исправь аналогичные ошибки.
Igor1 commented 3 years ago
Poster

Исправил.
e4e499fe61

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/e4e499fe6154029757251a9a82fc96bee3b2988b
}
else {
info_status_app("boxInfoMessOK");
imgInfo->set_from_icon_name("com.ublinux.ubl-settings-bootloader.checked", Gtk::ICON_SIZE_MENU);
Owner

Старайся все имена внешних файлов и прочего внешнего, выносить в переменные которые описывать в одном месте !

Старайся все имена внешних файлов и прочего внешнего, выносить в переменные которые описывать в одном месте !
Igor1 commented 3 years ago
Poster

Исправил.
e4e499fe61

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/e4e499fe6154029757251a9a82fc96bee3b2988b
}
void MainWindow::synopsis_show() {
string cmd = "xdg-open " + string(_("https://wiki.ublinux.com/ru/Программное_обеспечениерограммы_и_утилиты/Все/")) + app_name;
Owner

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

Аналогично вынести в переменную, и вызывать с подстановкой параметров если требуется !
Igor1 commented 3 years ago
Poster

Исправил.
2de795e080

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/2de795e0809cb7ef6b22d4ca20c7d11868c28cf1
source/util.cc Outdated
return vec_users;
}
string read_passwd(string username) {
Owner

Для чего ты парсишь /etc/shadow ?
Зачем тебе пароль ?

Для чего ты парсишь /etc/shadow ? Зачем тебе пароль ?
Igor1 commented 3 years ago
Poster

Исправил.
085c66ec44

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/085c66ec44ae2c31fa6eed915b18a3958c8a8ffe
source/util.cc Outdated
string cmd = "xdg-open " + str_link;
if (geteuid() == 0) {
string response_user = getlogin();
cmd = "su -l " + response_user + " -c \" DISPLAY=$DISPLAY " + cmd + " \"";
Owner

Аналогично внешний вызов, перенести в переменную, с подстановкой нужных параметров

Аналогично внешний вызов, перенести в переменную, с подстановкой нужных параметров
Igor1 commented 3 years ago
Poster

Исправил.
2de795e080

Исправил. https://gitea.ublinux.ru/Igor1/ubl-settings-bootloader/commit/2de795e0809cb7ef6b22d4ca20c7d11868c28cf1
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Owner

На указанные ошибки нет комментариев !

На указанные ошибки нет комментариев !
Igor1 added 3 commits 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Collaborator

handle = dlopen("libwebkit2gtk-4.0.so", RTLD_LAZY);
Предлагаю имя библиотеки модифицировать в регулярку, чтобы при смене версии либы, проверка всё так же проходила.

`handle = dlopen("libwebkit2gtk-4.0.so", RTLD_LAZY);` Предлагаю имя библиотеки модифицировать в регулярку, чтобы при смене версии либы, проверка всё так же проходила.
Owner

handle = dlopen("libwebkit2gtk-4.0.so", RTLD_LAZY);
Предлагаю имя библиотеки модифицировать в регулярку, чтобы при смене версии либы, проверка всё так же проходила.

Вообще убрать эту зависимость при компиляции !

> `handle = dlopen("libwebkit2gtk-4.0.so", RTLD_LAZY);` > Предлагаю имя библиотеки модифицировать в регулярку, чтобы при смене версии либы, проверка всё так же проходила. Вообще убрать эту зависимость при компиляции !
Igor1 added 3 commits 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Igor1 added 2 commits 3 years ago
Collaborator

Проверил (от последнего сообщения руководителя проекта).

Проверил (от последнего сообщения руководителя проекта).
Igor1 added 1 commit 3 years ago
Igor1 added 1 commit 3 years ago
Collaborator

Проверил.

Проверил.
asmeron merged commit 7e2cc70ac2 into develop 3 years ago
user-166 requested review from asmeron 2 years ago
asmeron approved these changes 1 year ago

Reviewers

asmeron approved these changes 1 year ago
The pull request has been merged as 7e2cc70ac2.
Sign in to join this conversation.
Loading…
There is no content yet.