master #4

Merged
asmeron merged 37 commits from YanTheKaller/ubl-settings-repomanager:master into master 2 years ago
Collaborator
There is no content yet.
YanTheKaller added 31 commits 2 years ago
YanTheKaller added 1 commit 2 years ago
YanTheKaller added 1 commit 2 years ago
YanTheKaller added 1 commit 2 years ago
asmeron requested changes 2 years ago
asmeron left a comment
Owner

Имя файла кривое: screenshot/screensgot.png

Имя файла кривое: screenshot/screensgot.png
Makefile Outdated
build: depend prepare
@echo "Build ..."; \
$(MAKE_COMMAND) --directory=${CMAKE_BUILD_DIR}; \
sed -r "s/^(string version_application).*/\1;/" -i ${FILE_VER}; \
Owner

Вернуть, это очищает версию после сборки !
В первую очередь вам удобнее.

Вернуть, это очищает версию после сборки ! В первую очередь вам удобнее.
Poster
Collaborator

Этот sed очищает версию перед сборкой дебаг версии, на 129 строке я убираю версию из заголовочника

Этот sed очищает версию перед сборкой дебаг версии, на 129 строке я убираю версию из заголовочника
Owner

Дебаг версия пускай будет без версии, дебаг версию через makefile собирайте, а не постоянно в CMakeLists.txt вставлять -g и каждый раз забывать его удалять, перед PR

Дебаг версия пускай будет без версии, дебаг версию через makefile собирайте, а не постоянно в CMakeLists.txt вставлять -g и каждый раз забывать его удалять, перед PR
Poster
Collaborator

-g не работает в cmake для С кода по какой-то причине

-g не работает в cmake для С кода по какой-то причине
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
Makefile Outdated
LATEST_TAG="Development"; \
fi; \
sed -r "s/^(string version_application).*/\1 = \"$${LATEST_TAG}\";/" -i ${FILE_VER}; \
sed -r "s/^(string version_application).*/\1=\"$${LATEST_TAG}\";/" -i ${FILE_VER}; \
Owner

Зачем пробелы убирать, как это отражается на функциональности ?

Зачем пробелы убирать, как это отражается на функциональности ?
Poster
Collaborator

это тоже к вопросу выше. Потому что в моей очистке sed-ом без пробелов версия записывается

это тоже к вопросу выше. Потому что в моей очистке sed-ом без пробелов версия записывается
Owner

Выше ответил, вернуть !

Выше ответил, вернуть !
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
@$(RM) "${DESTDIR}${PREFIX}/share/icons/hicolor/scalable/apps/com.ublinux.${PKGNAME}.svg"
@$(RM) "${DESTDIR}${PREFIX}/share/icons/hicolor/scalable/apps/com.ublinux.${PKGNAME}.checked.svg"
Owner

Убрать, т.к. есть цикл который удаляет все картинки. Да и путь не верный указал, устанавливает в /status/, а удаляешь из /apps/ !

Последующие то-же самое !

Убрать, т.к. есть цикл который удаляет все картинки. Да и путь не верный указал, устанавливает в /status/, а удаляешь из /apps/ ! Последующие то-же самое !
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
Makefile Outdated
@$(RM) -rd "${DESTDIR}${PREFIX}/share/${PKGNAME}/images/${PKGNAME}-banner.png"
@$(RM) -rd "${DESTDIR}${PREFIX}/share/${PKGNAME}"
@$(RM) "${DESTDIR}${PREFIX}/share/polkit-1/actions/com.ublinux.${PKGNAME}.policy"
@$(RM) "${DESTDIR}${PREFIX}/share/icons/hicolor/scalable/apps/com.ublinux.${PKGNAME}.document-green-symbolic.svg"
Owner

Убрать, т.к. есть цикл который удаляет все картинки. Да и путь не верный указал, устанавливает в /status/, а удаляешь из /apps/ !

Последующие то-же самое !

Убрать, т.к. есть цикл который удаляет все картинки. Да и путь не верный указал, устанавливает в /status/, а удаляешь из /apps/ ! Последующие то-же самое !
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
Makefile Outdated
[ -d "${DESTDIR}${PREFIX}/share/applications" ] && touch "${DESTDIR}${PREFIX}/share/applications" &>/dev/null || true; \
fi
@echo "Install: OK"
@sed -r "s/^(string version_application=\"*\").*/string version_application;/" -i source/${PKGNAME}.h; \
Owner

Убрать !

Убрать !
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
link_directories(${GTK_LIBRARY_DIRS})
add_definitions(${GTK_CFLAGS_OTHER})
#pkg_check_modules(VTE291 REQUIRED vte-2.91)
Owner

Если не нужен, то удалить !
Ниже аналогично !

Если не нужен, то удалить ! Ниже аналогично !
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
set(LIBRARIES
${GTK_LIBRARIES}
${WEBKIT_LIBRARIES}
# ${VTE291_LIBRARIES}
Owner

Удалить !

Удалить !
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
if (res==GTK_RESPONSE_ACCEPT){
char *filename = gtk_file_chooser_get_filename(GTK_FILE_CHOOSER(chooser));
FILE *file = popen(yon_char_unite("LC_ALL=EN pacman -Q --info --file ",filename," |grep -E \"Name|Version|Architecture\" |sed -e 's/ *//g'",NULL),"r");
Owner

Работу с консольными команды вынести из кода в define !
Касается всех утилит !

Работу с консольными команды вынести из кода в define ! Касается всех утилит !
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
#define glade_repo_path "/com/ublinux/ui/ubl-settings-repomanager-windows.glade"
#define banner_path "/com/ublinux/images/ubl-settings-repomanager-banner.png"
#define CssPath "/com/ublinux/css/ubl-settings-repomanager.css"
#define config_path yon_char_unite("/home/",yon_ubl_root_user_get(),"/.config/",LocaleName,"/",LocaleName,".conf",NULL)
Owner

Получать из глобальных переменных домашний каталог пользователя, может быть и не /home/

Получать из глобальных переменных домашний каталог пользователя, может быть и не /home/
Poster
Collaborator
https://gitea.ublinux.ru/Applications/ubl-settings-repomanager/commit/5ec52c4b928bdc4fc2c052ba7fe6bd4695d45df9
YanTheKaller added 1 commit 2 years ago
asmeron requested changes 2 years ago
void yon_add_file(main_window *widgets){
GtkFileFilter *file_filter = gtk_file_filter_new();
gtk_file_filter_add_pattern(file_filter,"*.pkg.tar.zst");
Owner

Расширение пакета не обязательно *.pkg.tar.zst, заменить на *.pkg.tar.*

Везде ниже аналогично !

Расширение пакета не обязательно \*.pkg.tar.zst, заменить на \*.pkg.tar.\* Везде ниже аналогично !
asmeron reviewed 2 years ago
@install -Dm644 -t "${DESTDIR}${PREFIX}/share/polkit-1/actions/" "com.ublinux.${PKGNAME}.policy"
@install -Dm644 -t "${DESTDIR}${PREFIX}/share/icons/hicolor/scalable/status/" "com.ublinux.${PKGNAME}.checked.svg"
@install -Dm644 -t "${DESTDIR}${PREFIX}/share/icons/hicolor/scalable/status/" "com.ublinux.${PKGNAME}.warning.svg"
@install -dm755 "${DESTDIR}${PREFIX}/share/polkit-1/actions"
Owner

Удалить !

Удалить !
YanTheKaller added 1 commit 2 years ago
YanTheKaller added 1 commit 2 years ago
asmeron merged commit 56166cb88b into master 2 years ago

Reviewers

asmeron requested changes 2 years ago
The pull request has been merged as 56166cb88b.
Sign in to join this conversation.
Loading…
There is no content yet.