sailfishos-patches / patchmanager

Patchmanager for SailfishOS
https://openrepos.net/content/patchmanager/patchmanager
Other
21 stars 22 forks source link

[Bug] files paths in the patch not always correctly parsed #445

Open robang74 opened 1 year ago

robang74 commented 1 year ago

SailFish OS version: 4.5.0.9 Patchmanger Version: 3.2.9 Hardware: Xperia 10 II

BUG DESCRIPTION

Referring to this patch (untested code):

this is the output in the log when the patch is installed:

void PatchManagerObject::doRefreshPatchList() Collecting info for:  "zram-swap-resize-script"
void PatchManagerObject::doRefreshPatchList() Possible conflict in:  "/usr/bin/zram_swap_resize.sh"

and this the same when it is not installed:

void PatchManagerObject::doRefreshPatchList() Collecting info for:  "zram-swap-resize-script"
void PatchManagerObject::doRefreshPatchList() Possible conflict in:  "/zram_swap_resize.sh"

in the second case the patch files paths parsing is wrong, unless I created that file which does not exist by default.

STEPS TO REPRODUCE

  1. download from Web Catalog the patch above
  2. switch the Patch Manager to developer mode
  3. then enable and disable the patch
  4. check the Patch Manager log

ADDITIONAL INFORMATION

Proposed bug-fix

Let me know and I will do a pull/merge request.

CODeRUS commented 1 year ago

Please take care to fix the code issues reported by this person, but create your own changes. no commits authored by this person are allowed to be merged in this repo.

CODeRUS commented 1 year ago

@nephros @nephros @Olf0 fyi

Olf0 commented 1 year ago

@b100dian was missed.

Olf0 commented 1 year ago

Please let us carry out this meta-discussion at the oldest bug report filed by robang74: #444

Every thing discussed there an the meta-level is sure applicable to the filed issues #445 and #446, too (plus any future ones by robang74).