Skip to content

Commit 4b4c37e

Browse files
committed
Packages (Linux): adds AppImage packages support
Fixes #2179
1 parent 8970774 commit 4b4c37e

6 files changed

Lines changed: 35 additions & 2 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ Features:
1414
* Improves PhysicalMemory reliability on OpenBSD and FreeBSD, including support for x86 UEFI systems on OpenBSD and non-UEFI systems on FreeBSD (PhysicalMemory, OpenBSD / FreeBSD)
1515
* Adds `display.common.(ndigits|spaceBeforeUnit)` options to configure common formatting for duration, percent, size, frequency and temperature values (#2234)
1616
* Slightly improves performance by skipping probes for ignored device types (Sound)
17+
* Adds support for AppImage package detection (#2179, Packages, Linux)
1718

1819
Bugfixes:
1920
* Fixes a potential crash on certain Intel GPU models when using `--gpu-driver-specific` (#2259, GPU, Linux)

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ if(NOT BINARY_LINK_TYPE IN_LIST BINARY_LINK_TYPE_OPTIONS)
119119
message(FATAL_ERROR "BINARY_LINK_TYPE must be one of ${BINARY_LINK_TYPE_OPTIONS}")
120120
endif()
121121

122-
set(PACKAGE_MANAGERS AM APK BREW CHOCO DPKG EMERGE EOPKG FLATPAK GUIX LINGLONG LPKG LPKGBUILD MACPORTS NIX OPKG PACMAN PACSTALL PALUDIS PISI PKG PKGTOOL RPM SCOOP SNAP SOAR SORCERY WINGET XBPS)
122+
set(PACKAGE_MANAGERS AM APPIMAGE APK BREW CHOCO DPKG EMERGE EOPKG FLATPAK GUIX LINGLONG LPKG LPKGBUILD MACPORTS NIX OPKG PACMAN PACSTALL PALUDIS PISI PKG PKGTOOL RPM SCOOP SNAP SOAR SORCERY WINGET XBPS)
123123
foreach(package_manager ${PACKAGE_MANAGERS})
124124
if(package_manager STREQUAL "WINGET")
125125
option(PACKAGES_DISABLE_${package_manager} "Disable ${package_manager} package manager detection by default" ON)

src/detection/packages/packages.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
typedef struct FFPackagesResult {
77
uint32_t amSystem;
88
uint32_t amUser;
9+
uint32_t appimage;
910
uint32_t apk;
1011
uint32_t brew;
1112
uint32_t brewCask;

src/detection/packages/packages_linux.c

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,25 @@ static uint32_t countFilesRecursive(FFstrbuf* baseDir, const char* dirname, cons
118118
return sum;
119119
}
120120

121+
static uint32_t getNumElementsBySuffix(FFstrbuf* baseDir, const char* dirname, const char* suffix) {
122+
uint32_t baseDirLength = baseDir->length;
123+
ffStrbufAppendS(baseDir, dirname);
124+
FF_AUTO_CLOSE_DIR DIR* dirp = opendir(baseDir->chars);
125+
ffStrbufSubstrBefore(baseDir, baseDirLength);
126+
if (dirp == NULL) {
127+
return 0;
128+
}
129+
130+
uint32_t count = 0;
131+
struct dirent* entry;
132+
while ((entry = readdir(dirp)) != NULL) {
133+
if (entry->d_name[0] != '.' && ffStrEndsWithIgnCase(entry->d_name, suffix)) {
134+
++count;
135+
}
136+
}
137+
return count;
138+
}
139+
121140
static uint32_t getXBPSImpl(FFstrbuf* baseDir) {
122141
DIR* dir = opendir(baseDir->chars);
123142
if (dir == NULL) {
@@ -621,4 +640,9 @@ void ffDetectPackagesImpl(FFPackagesResult* result, FFPackagesOptions* options)
621640
if (!(options->disabled & FF_PACKAGES_FLAG_SOAR_BIT)) {
622641
result->soar += getSQLite3Int(&baseDir, ".local/share/soar/db/soar.db", "SELECT COUNT(DISTINCT pkg_id || pkg_name) FROM packages WHERE is_installed = true", "soar");
623642
}
643+
644+
if (!(options->disabled & FF_PACKAGES_FLAG_APPIMAGE_BIT)) {
645+
result->appimage += getNumElementsBySuffix(&baseDir, "/AppImages", ".appimage");
646+
result->appimage += getNumElementsBySuffix(&baseDir, "/Applications", ".appimage");
647+
}
624648
}

src/modules/packages/option.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ typedef enum FF_A_PACKED FFPackagesFlags {
3737
FF_PACKAGES_FLAG_SOAR_BIT = 1ULL << 30,
3838
FF_PACKAGES_FLAG_KISS_BIT = 1ULL << 31,
3939
FF_PACKAGES_FLAG_MOSS_BIT = 1ULL << 32,
40+
FF_PACKAGES_FLAG_APPIMAGE_BIT = 1ULL << 33,
4041
FF_PACKAGES_FLAG_FORCE_UNSIGNED = UINT64_MAX,
4142
} FFPackagesFlags;
4243
static_assert(sizeof(FFPackagesFlags) == sizeof(uint64_t), "");

src/modules/packages/packages.c

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ bool ffPrintPackages(FFPackagesOptions* options) {
109109
FF_PRINT_PACKAGE_NAME(amSystem, "am")
110110
FF_PRINT_PACKAGE_NAME(amUser, "appman")
111111
}
112+
FF_PRINT_PACKAGE(appimage)
112113
FF_PRINT_PACKAGE(sorcery)
113114
FF_PRINT_PACKAGE(lpkg)
114115
FF_PRINT_PACKAGE(lpkgbuild)
@@ -166,6 +167,7 @@ bool ffPrintPackages(FFPackagesOptions* options) {
166167
FF_ARG(counts.pacstall, "pacstall"),
167168
FF_ARG(counts.mport, "mport"),
168169
FF_ARG(counts.amUser, "am-user"),
170+
FF_ARG(counts.appimage, "appimage"),
169171
FF_ARG(counts.pkgsrc, "pkgsrc"),
170172
FF_ARG(counts.hpkgSystem, "hpkg-system"),
171173
FF_ARG(counts.hpkgUser, "hpkg-user"),
@@ -220,8 +222,9 @@ void ffParsePackagesJsonObject(FFPackagesOptions* options, yyjson_val* module) {
220222
case 'A':
221223
if (false)
222224
;
223-
FF_TEST_PACKAGE_NAME(APK)
224225
FF_TEST_PACKAGE_NAME(AM)
226+
FF_TEST_PACKAGE_NAME(APPIMAGE)
227+
FF_TEST_PACKAGE_NAME(APK)
225228
break;
226229
case 'B':
227230
if (false)
@@ -352,6 +355,7 @@ void ffGeneratePackagesJsonConfig(FFPackagesOptions* options, yyjson_mut_doc* do
352355
if (false)
353356
;
354357
FF_TEST_PACKAGE_NAME(AM)
358+
FF_TEST_PACKAGE_NAME(APPIMAGE)
355359
FF_TEST_PACKAGE_NAME(APK)
356360
FF_TEST_PACKAGE_NAME(BREW)
357361
FF_TEST_PACKAGE_NAME(CHOCO)
@@ -407,6 +411,7 @@ bool ffGeneratePackagesJsonResult(FF_A_UNUSED FFPackagesOptions* options, yyjson
407411
FF_APPEND_PACKAGE_COUNT(all)
408412
FF_APPEND_PACKAGE_COUNT(amSystem)
409413
FF_APPEND_PACKAGE_COUNT(amUser)
414+
FF_APPEND_PACKAGE_COUNT(appimage)
410415
FF_APPEND_PACKAGE_COUNT(apk)
411416
FF_APPEND_PACKAGE_COUNT(brew)
412417
FF_APPEND_PACKAGE_COUNT(brewCask)
@@ -507,6 +512,7 @@ FFModuleBaseInfo ffPackagesModuleInfo = {
507512
{ "Number of linglong packages", "linglong" },
508513
{ "Number of pacstall packages", "pacstall" },
509514
{ "Number of mport packages", "mport" },
515+
{ "Number of appimage packages", "appimage" },
510516
{ "Number of am-user (aka appman) packages", "am-user" },
511517
{ "Number of pkgsrc packages", "pkgsrc" },
512518
{ "Number of hpkg-system packages", "hpkg-system" },

0 commit comments

Comments
 (0)