The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

Выпуск интегрированной среды разработки Qt Creator 4.3.0 , opennews (??), 25-Май-17, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


23. "Выпуск интегрированной среды разработки Qt Creator 4.3.0 "  +/
Сообщение от anonymous (??), 25-Май-17, 12:26 
Добавлять в дерево файлов != добавлять к сборке.

Собственно в своих модификациях я это и сделал:
1. отображаются все файлы из директории проекта
2. добавления/удаление/переименование активировано, но ответственность за добавление к самому CMake - на пользователе. Или глоббинг (что часто не есть хорошо).

Да, эти модицикации противоречат "духу" Project View в QtC (можно найти подтверждения в их рассылке), поэтому никогда не будут приняты в апстрим :)

Ответить | Правка | Наверх | Cообщить модератору

26. "Выпуск интегрированной среды разработки Qt Creator 4.3.0 "  +/
Сообщение от максemail (??), 25-Май-17, 14:38 
"Добавлять к сборке" = выбрать target. Сказал строить что-то, и оно строится. Не сказал, не строится. Но в проекте висит.
Ответить | Правка | Наверх | Cообщить модератору

27. "Выпуск интегрированной среды разработки Qt Creator 4.3.0 "  +/
Сообщение от nibemail (?), 25-Май-17, 14:53 
Ну если вы предложите патч с рабочей версией, её не будут отклонять только потому, что она противоречит чьим-то идеалам. Добавления файлов до сих пор нет, потому что никто не сделал, а не из-за конспирологии
Ответить | Правка | К родителю #23 | Наверх | Cообщить модератору

39. "Выпуск интегрированной среды разработки Qt Creator 4.3.0 "  +/
Сообщение от анонимус (??), 26-Май-17, 03:50 
Вы, видно, не часто принимали участие в проектах. Я не патч, я ревью в геррите завёл, благо у меня уже несколько комитов в QtC есть. Его и забородили с таким обоснованием. Некоторые сопутствующие изменения пропихнуть таки получилось, так что на текущий момент поддерживаемая кодовая база ощутимо уменьшилась.
Ответить | Правка | Наверх | Cообщить модератору

40. "Выпуск интегрированной среды разработки Qt Creator 4.3.0 "  +1 +/
Сообщение от nibemail (?), 26-Май-17, 11:06 
https://codereview.qt-project.org/#/c/180827/ этот патч?

Там довольно развёрнуто вам ответили.
Со своей стороны могу дополнить. Люди не хотят затаскивать полу-рабочие решения. Или решения которые не будут работать на определённых конфигурациях.
Отображать файлы как видит билд система мб спорное решение. Но если отображать всё, то, для примера, ваш проект утонет в сборочных артефактах, содержимом gitignore или весь экран будет занимать внутренности .git. Гит только для примера. Как в таком случае семантично поделить сущности на сорсы из проекта, а потом включить для них функционал, затрудняюсь сказать.

На счёт добавления файлов, просмотрел в вашем сообщении "ответственность за добавление к самому CMake - на пользователе".
Если делать, то как раз с редактированием проектных файлов в первую очередь. Если делать как вы предлагаете, то креатор утонет в багрепортах(добавили файл, ничего не компилится) и разрабам ничего не останется, только как ревертнуть ваш коммит.

Пс патч/коммит/etc к словам можно было бы не придираться и не заниматься писькомерством

Ответить | Правка | Наверх | Cообщить модератору

44. "Выпуск интегрированной среды разработки Qt Creator 4.3.0 "  +/
Сообщение от dhamp (?), 27-Май-17, 00:02 
>Отображать файлы как видит билд система мб спорное решение.

Если CMakeLists.txt не содержит явного списка файлов(список файлов генерируется, часть файлов должна собираться только на некотором подмножестве поддерживаемых платформ)
Что предлагается ?
>ваш проект утонет в сборочных артефактах, содержимом gitignore или весь экран будет занимать внутренности .git.

по поводу артифактов - out-of-source-build придумали как раз для того чтобы не срать разной генерируемой дичью в дерево исходников
содержимое gitignore - а файлы перечисленные в нём могут быть текстовыми и их вполне можно редактировать
cmakeprojectmanager2 внутренности .git директории не показывал.

>Если делать, то как раз с редактированием проектных файлов в первую очередь.

Не всем это нужно, особенно если CMakeLists.txt не тривиальны.

Ответить | Правка | Наверх | Cообщить модератору

46. "Выпуск интегрированной среды разработки Qt Creator 4.3.0 "  –1 +/
Сообщение от nibemail (?), 27-Май-17, 01:26 
> часть файлов должна собираться только на некотором подмножестве поддерживаемых платформ

аналогично с qmake проектами: отображать все возможные варианты, без учета условий

> по поводу артифактов - out-of-source-build придумали как раз для того чтобы не срать разной генерируемой дичью в дерево исходников

но это не логично ломать у одного пользователя, чтобы добавить фичу для второго

> cmakeprojectmanager2 внутренности .git директории не показывал.

Круто а как у него с MySuperCoolDvcs, игнорируется только всё что начинается с точки?

> Не всем это нужно, особенно если CMakeLists.txt не тривиальны.

Опять же странная логика, если кому-то не нужно, то давайте ломать всем.
Следуя этой логике, намного логичнее не использовать cmake, например, а использовать любую другую систему сборки. В qmake проекты можно добавлять файлы, значит не используйте cmake.:/ Или еще хороший пример, почему вы не листите файлы через Ctrl+O -> диалог выбора файла, там можно даже вид вьюх настроить. Не говоря уже о том, что есть локатор с фильтрами.

Надеюсь хотя бы минимально получилось донести, что это плохая идея добавлять функционал, который ломает что-либо работающее. Или делает что-то неоднородно с существующим функционалом.

Ответить | Правка | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру