Summary: | krita includes nonfree image lena*.png | ||
---|---|---|---|
Product: | [Applications] krita | Reporter: | Rex Dieter <rdieter> |
Component: | General | Assignee: | Krita Bugs <krita-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | dimula73, griffinvalley, halla |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Rex Dieter
2016-02-21 12:15:52 UTC
Yeah, we've noticed. We're in a busy moment and that's a hella lot of tests that rely on it, so we're trying to figure out a spot in our schedule when to tackle it. We're not even sure how it got in 4 whole times... Alright... UI uses her for... filter_stroke_test.cpp kis_prescaled_projection_test.cpp Tools uses her for... move_stroke_test.cpp filters uses her for kis_all_filter_test.cpp kis_crash_filter_test.cpp SDK... doesn't use her at all... Git commit 1d474794bf2503c0b23e70418a2785f6e8def779 by Wolthera van Hövell. Committed on 21/02/2016 at 16:38. Pushed by woltherav into branch 'master'. Remove lena.png from sdk. 1/4 lenas removed. I don't know enough about tests to dare tackling the others, but this one seemed easy enough. D +- -- sdk/tests/data/lena.png http://commits.kde.org/krita/1d474794bf2503c0b23e70418a2785f6e8def779 Hi, Wolthera! The images in sdk/tests/data were my attempt not to store the data in every testing directory and fetch them from a centralized storage via TestUtil::fetchDataFileLazy(). According to grep lena.png was not used this way, so the commit is safe. The rest of the usages weem to be not very difficult to fix libs/ui/tests/filter_stroke_test.cpp: QImage src(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); libs/ui/tests/kis_animation_importer_test.cpp: files.append(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); libs/ui/tests/kis_animation_importer_test.cpp: QImage source2(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); libs/ui/tests/kis_prescaled_projection_test.cpp: QImage qimage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); libs/ui/tests/kis_prescaled_projection_test.cpp: QImage qimage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); libs/ui/tests/kis_prescaled_projection_test.cpp: QImage referenceImage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); libs/ui/tests/kis_prescaled_projection_test.cpp: sourceImage = QImage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); plugins/filters/tests/kis_all_filter_test.cpp: QImage qimage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); plugins/filters/tests/kis_all_filter_test.cpp: QImage qimage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); plugins/filters/tests/kis_all_filter_test.cpp: QImage qimage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); plugins/filters/tests/kis_all_filter_test.cpp: QImage qimage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); plugins/filters/tests/kis_crash_filter_test.cpp: QImage qimage(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); plugins/tools/basictools/tests/move_stroke_test.cpp: QImage src(QString(FILES_DATA_DIR) + QDir::separator() + "lena.png"); Fixed now, in 8fc80f0360dc1ee0c8ce48abcb9f7013917780a6 and 134a00200eeda0b3d5d3ee9b598879a94e48797c |