master #11
Merged
asmeron
merged 110 commits from Igor1/ubl-settings-bootloader:master into develop 3 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'Igor1/ubl-settings-bootloader:master'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Частично сделано сохранение. Параметры "Логин защиты", "Пароль защиты","Загрузка по умолчанию" не работают.
Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h
DEPENDS = /bin/cmakePREFIX ?= /usrPKGNAME = $(MAKEFILE_DIR)#PKGNAME = ubl-settings-datetimeubl-settings-datetime ??
Исправил
d54cae5d80init:@echo "Initialize ..."; \sed -r "s/(version_application = ).*/\1\"$$(grep 'VERSION' ${FILE_VERSION} | cut -d" " -f2)\";/" -i source/ubl-settings-datetime.cc; \ubl-settings-datetime ??
Исправил
d54cae5d80<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>"Загрузка системы" ??
Что не так ?
В тз "Загрузка системы"
2d72e52ffdЗаменил на "ubl-settings-bootloader"
string response = "";string key = "";string value = "";cmd = "ubconfig --default --source " + str_flag_load + " get " + this->sections;Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h
У меня все грузиться одной командой. У меня универсальное решение которое можно использовать с минимальными затратами времени, есть возможность масштабирования. Выносить все в #define когда команду можно генерировать исходя из параметров зачем ? Мы же не на 1 курсе. Из-за того что я использую словари у меня скорость выполнения поиска по словарю Min(O(1)) -> Max(O(n)). Когда я спросил у Вани какое у тебя О больше, он сказал я не знаю что это.
Игорь, консольные команды это не часть языка С++, это внешние вызовы, их удобнее в случае проблем или изменений править в одном месте.
Значения так-же подстановкой передаваемым преременным можно указывать, вот и получитаться, что ты генерируешь изменения которые предеаются в заранее сконструированный синтаксис вызова внешней команды.
Это никак не влияет на модульность или KISS. Это только влияет на стиль и удобочитаемость. Большой + что команды в случае изменения легко подправить, не погружаясь в чересчур лишнюю генерацию !
Либо предложи компромисс, между как у Ивана и как у тебя !
Я к тому что в одном проекте 5 команд в другом 10 и каждый раз править код ?
Есть вариант, ключи словаря они же параметры ubconfig вынести в отдельный заголовочный файл.
Почему не сделать, как делает Иван ?
Я не вижу ни одного минуса !
У Ивана самописанные словари, зачем мне с++ изобретать велосипеды, когда они уже есть в С++ ?
Исправил.
65f13c4662value = (*map_gui)[key];if (this->check_save(str_flag_save, key)) {if (value.length() != 0) {cmd = "ubconfig --target " + str_flag_save + " set " + sections + " " +Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h
Я стараюсь развивать модульность, универсальность , а не фигачить все в два файла. Пытаюсь следовать методике KISS не всегда получается, но я стараюсь
Исправил.
65f13c4662key + "=\"" + value + "\"";}else if (value.length() == 0) {cmd = "ubconfig --target " + str_flag_save + " remove " + sections + " " + key;Посмотреть как делает Иван, и все консольные команды вынести в переменные #define и в .h
Я стараюсь развивать модульность, универсальность , а не фигачить все в два файла. Пытаюсь следовать методике KISS не всегда получается, но я стараюсь
Исправил.
65f13c4662# SOME DESCRIPTIVE TITLE.Реквизиты сделать правильные
Исправил
d54cae5d80# Russian translations for PACKAGE package.Реквизиты сделать правильные
Исправил
d54cae5d80treeViewIPT->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);Имена внешних файлов так-же в переменные !
Исправить везде по тексту !
Исправил.
7ba4533fc0btnLoad->set_label(_("Load"));this->set_title(_("ubl-settings-bootloader"));// Todo: checkentrySecurityLogin->set_tooltip_text(gettext("User name to enter boot configuration editing mode"));Нет единого стиля, если используешь для локализации функцию _ то и используй её, зачем то так то сяк gettext ?
Заменил.
412495a222bool 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);Старайся все имена внешних файлов и прочего внешнего, выносить в переменные которые описывать в одном месте !
Везде по тексту исправь аналогичные ошибки.
Исправил.
e4e499fe61}else {info_status_app("boxInfoMessOK");imgInfo->set_from_icon_name("com.ublinux.ubl-settings-bootloader.checked", Gtk::ICON_SIZE_MENU);Старайся все имена внешних файлов и прочего внешнего, выносить в переменные которые описывать в одном месте !
Исправил.
e4e499fe61}void MainWindow::synopsis_show() {string cmd = "xdg-open " + string(_("https://wiki.ublinux.com/ru/Программное_обеспечение/Программы_и_утилиты/Все/")) + app_name;Аналогично вынести в переменную, и вызывать с подстановкой параметров если требуется !
Исправил.
2de795e080return vec_users;}string read_passwd(string username) {Для чего ты парсишь /etc/shadow ?
Зачем тебе пароль ?
Исправил.
085c66ec44string cmd = "xdg-open " + str_link;if (geteuid() == 0) {string response_user = getlogin();cmd = "su -l " + response_user + " -c \" DISPLAY=$DISPLAY " + cmd + " \"";Аналогично внешний вызов, перенести в переменную, с подстановкой нужных параметров
Исправил.
2de795e080На указанные ошибки нет комментариев !
handle = dlopen("libwebkit2gtk-4.0.so", RTLD_LAZY);Предлагаю имя библиотеки модифицировать в регулярку, чтобы при смене версии либы, проверка всё так же проходила.
Вообще убрать эту зависимость при компиляции !
Проверил (от последнего сообщения руководителя проекта).
Проверил.
7e2cc70ac2into develop 3 years agoReviewers
7e2cc70ac2.