search

ugeneunipro / ugene

UGENE is free open-source cross-platform bioinformatics software
http://ugene.net
GNU General Public License v2.0
207 stars 60 forks source link

UGENE-5239 "~/" or nonexistent directory for Make Blast Database output path #1627

Closed rasputinkirill closed 3 months ago

rasputinkirill commented 3 months ago

Не стал делать поддержку "~" как валидной директории так как у нас нигде нет явного распознавания ее как домашней директории пользователя.

rasputinkirill commented 3 months ago

На винде проверка на только чтение через QFileInfo::isWritable() не работает правильно, поэтому сделал так.

yalgaer commented 3 months ago

На винде проверка на только чтение через QFileInfo::isWritable() не работает правильно, поэтому сделал так.

У нас же есть утилита уже используемая в десятке мест которая проверяет можно ли записать или нет по данному пути. Она не подошла?

rasputinkirill commented 3 months ago

Работает с FileAndDirectoryUtils::isDirectoryWritable(const QString& dirPath)

rasputinkirill commented 3 months ago

Добавил проверку в таску, дополнил тест. Обращаю внимание, что путь переданный в таску создается инструментом makeblstdb самомтоятельно поэтому проверку на существования указанного пути в таске не произвожу, только на возможность записи.

yalgaer commented 3 months ago

Замечаний нет, апрув после тестов