termux / termux-packages

A package build system for Termux.
https://termux.dev
Other
13.28k stars 3.06k forks source link

[Package]: filelight #21401

Closed EngineerDragon closed 1 week ago

EngineerDragon commented 2 months ago

Why is it worth to add this package?

The requested package is required for me in order to clean up my unwanted extra files from my termux workspace. This package provides me proper visualisation for my big files.

Home page URL

https://apps.kde.org/filelight/

Source code URL

https://github.com/KDE/filelight

Packaging policy acknowledgement

Additional information

I managed to build the package. Unfortunately I have encountered an error while building kirigami due to an ambigous error. When I ran it It told me it is unable to find the required qml modules provided by kconfig even though kf6-kconfig package is installed.

Error occured during builing kirigami


/data/data/com.termux/files/home/kirigami/src/kirigamiplugin.cpp:73:50: error: use of undeclared identifier 'baseUrl'
   73 |    Kirigami::Platform::StyleSelector::setBaseUrl(baseUrl());
      |                                                  ^
``` If this gets fixed I assume filelight will not give error and work properly
Biswa96 commented 2 months ago

KDE apps require many dependencies which will flood the termux-packages repository. I am not sure if maintainers would accept that.

twaik commented 2 months ago

Most of these dependencies are reusable so probably yes. We should ask @sylirre, @truboxl and @licy183 about that.

licy183 commented 2 months ago

Adding KDE-related packages is OK for me. Actually, Termux already has lots of them. It would be better to add them in one PR, and use squash and merge.

Biswa96 commented 2 months ago

I like to do it in "tier" basis, like upstream does https://invent.kde.org/packaging/craft-blueprints-kde/-/tree/master/kde/frameworks?ref_type=heads. There could be too many packages.