]> git.ipfire.org Git - pakfire.git/commitdiff
package: Do not export the package Id
authorMichael Tremer <michael.tremer@ipfire.org>
Tue, 27 Apr 2021 11:39:39 +0000 (11:39 +0000)
committerMichael Tremer <michael.tremer@ipfire.org>
Tue, 27 Apr 2021 11:39:39 +0000 (11:39 +0000)
Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
src/_pakfire/package.c
src/libpakfire/include/pakfire/package.h
src/libpakfire/libpakfire.sym
src/libpakfire/package.c

index f5e1d57a5b044f48b7f7fd94524bbb99ec404c9e..022305b0bb3295acd2690ee0cbb8c64aa91c11b2 100644 (file)
@@ -76,15 +76,10 @@ static int Package_init(PackageObject* self, PyObject* args, PyObject* kwds) {
        return 0;
 }
 
-static long Package_hash(PackageObject* self) {
-       return pakfire_package_id(self->package);
-}
-
 static PyObject* Package_repr(PackageObject* self) {
        char* nevra = pakfire_package_get_nevra(self->package);
 
-       PyObject* repr = PyUnicode_FromFormat("<_pakfire.Package object id %ld, %s>",
-               Package_hash(self), nevra);
+       PyObject* repr = PyUnicode_FromFormat("<_pakfire.Package %s>", nevra);
        free(nevra);
 
        return repr;
@@ -1115,7 +1110,6 @@ PyTypeObject PackageType = {
        tp_doc:             "Package object",
        tp_methods:         Package_methods,
        tp_getset:          Package_getsetters,
-       tp_hash:            (hashfunc)Package_hash,
        tp_repr:            (reprfunc)Package_repr,
        tp_str:             (reprfunc)Package_str,
        tp_richcompare:     (richcmpfunc)Package_richcompare,
index 655fd4115895277efe54fa6c31b8bd6eb9fb3f49..671ba8ec2d62d5a82af9b412d30f4bc586dbe84e 100644 (file)
@@ -23,8 +23,6 @@
 
 #include <time.h>
 
-#include <solv/pooltypes.h>
-
 #include <pakfire/filelist.h>
 #include <pakfire/relation.h>
 #include <pakfire/relationlist.h>
@@ -41,8 +39,6 @@ int pakfire_package_identical(PakfirePackage pkg1, PakfirePackage pkg2);
 int pakfire_package_cmp(PakfirePackage pkg1, PakfirePackage pkg2);
 int pakfire_package_evr_cmp(PakfirePackage pkg1, PakfirePackage pkg2);
 
-Id pakfire_package_id(PakfirePackage pkg);
-
 char* pakfire_package_get_nevra(PakfirePackage pkg);
 const char* pakfire_package_get_name(PakfirePackage pkg);
 void pakfire_package_set_name(PakfirePackage pkg, const char* name);
@@ -155,8 +151,11 @@ enum pakfire_package_dump_flags {
 
 #ifdef PAKFIRE_PRIVATE
 
+#include <solv/pooltypes.h>
+
 PakfirePackage pakfire_package_create_from_solvable(Pakfire pakfire, Id id);
 
+Id pakfire_package_id(PakfirePackage pkg);
 int pakfire_package_is_source(PakfirePackage pkg);
 
 #endif
index 276ab96746596f414ec11df0648a885f342451bd..870664323a74512ee8ff4c3969c9b8b395080acd 100644 (file)
@@ -197,7 +197,6 @@ global:
        pakfire_package_get_uuid;
        pakfire_package_get_vendor;
        pakfire_package_get_version;
-       pakfire_package_id;
        pakfire_package_is_installed;
        pakfire_package_join_evr;
        pakfire_package_ref;
index 6553999a4063b6fca061dead5e033233d6655666..9a6b7e4cd5561168b895b45f081a7fe3652cbecd 100644 (file)
@@ -187,7 +187,7 @@ PAKFIRE_EXPORT int pakfire_package_evr_cmp(PakfirePackage pkg1, PakfirePackage p
        return pool_evrcmp(pool, s1->evr, s2->evr, EVRCMP_COMPARE);
 }
 
-PAKFIRE_EXPORT Id pakfire_package_id(PakfirePackage pkg) {
+Id pakfire_package_id(PakfirePackage pkg) {
        return pkg->id;
 }