pCompress bugs

Post here if you feel others can duplicate your so discovered "bug"

Moderator: Forum moderators

Post Reply
User avatar
MochiMoppel
Posts: 1236
Joined: Mon Jun 15, 2020 6:25 am
Location: Japan
Has thanked: 21 times
Been thanked: 439 times

pCompress bugs

Post by MochiMoppel »

I found pCompress as an option in the right-click menu of @radky 's BookwormPup64 10.0.7.
Never used it and thought to give it a try. It failed.

1) right-clicked the folder /boot
2) a dialog appears. I accepted all defaults and pressed "Create"
3) an error message appears
4) an empty archive /boot.tar.gz is created

Looking at the code it is obvious why the tool failed. inputdir=${inputfile%/*} in line 28 strips the inputfile /boot to an empty string instead of stripping to the parent directory '/'.
Even if this line would be fixed, the next command in line 29 [[ $inputdir == */ ]] && inputdir=${inputdir%\/} would "unfix" the error. This command is redundant anyway and shouldn't exist in the first place.

I would prefer to leave corrections to the author, but who is the author? And where is the forum thread? There is only a tersed "#2014" header comment in the file. IMHO this is not acceptable. I don't care for the license notice but I'm reluctant to use a tool of unknown origin.

radky
Posts: 355
Joined: Fri May 28, 2021 2:14 am
Has thanked: 41 times
Been thanked: 320 times

Re: pCompress bugs

Post by radky »

@MochiMoppel l

It seems pCompress was added to Puppy by the following Woof-CE commit (16 Oct 2019):

https://github.com/puppylinux-woof-CE/w ... afa8e159b2

ozsouth
Posts: 1569
Joined: Sun Jul 12, 2020 2:38 am
Location: S.E. Australia
Has thanked: 241 times
Been thanked: 704 times

Re: pCompress bugs

Post by ozsouth »

@MochiMoppel - user @plinej began working on this last year. v0.5 was released in Feb2024, but I notice that lines 62-63 are same as those you queried - see: https://forum.puppylinux.com/viewtopic. ... ss#p111780
I use the older version just to make hybrid iso's, which works.

Last edited by ozsouth on Mon Sep 09, 2024 1:06 am, edited 1 time in total.
User avatar
MochiMoppel
Posts: 1236
Joined: Mon Jun 15, 2020 6:25 am
Location: Japan
Has thanked: 21 times
Been thanked: 439 times

Re: pCompress bugs

Post by MochiMoppel »

radky wrote: Mon Sep 09, 2024 12:34 am

It seems pCompress was added to Puppy by the following Woof-CE commit (16 Oct 2019):

You mean it is a contribution of the same guy who left Woof-CE in December 2019 with the comment "Puppy users are retarded and stupid.I'm sick of puppy users, I'm going somewhere else where I don't to hear anything about pups and dogs and anything in between."?
Seriously? :lol:

ozsouth
Posts: 1569
Joined: Sun Jul 12, 2020 2:38 am
Location: S.E. Australia
Has thanked: 241 times
Been thanked: 704 times

Re: pCompress bugs

Post by ozsouth »

Always struck me as strange that that guy contributed so much code, but didn't appear (under that name at least) to be on the forum.
Meanwhile, plinej has made some very good utilities & may be able to sort this out.

User avatar
bigpup
Moderator
Posts: 6993
Joined: Tue Jul 14, 2020 11:19 pm
Location: Earth, South Eastern U.S.
Has thanked: 911 times
Been thanked: 1528 times

Re: pCompress bugs

Post by bigpup »

MochiMoppel wrote: Mon Sep 09, 2024 12:59 am
radky wrote: Mon Sep 09, 2024 12:34 am

It seems pCompress was added to Puppy by the following Woof-CE commit (16 Oct 2019):

You mean it is a contribution of the same guy who left Woof-CE in December 2019 with the comment "Puppy users are retarded and stupid.I'm sick of puppy users, I'm going somewhere else where I don't to hear anything about pups and dogs and anything in between."?
Seriously? :lol:

I would look at all the lines of code in that program and make sure nothing is doing anything bad.

With that attitude, who knows what may be lurking in it.

The things you do not tell us, are usually the clue to fixing the problem.
When I was a kid, I wanted to be older.
This is not what I expected :o

dimkr
Posts: 2423
Joined: Wed Dec 30, 2020 6:14 pm
Has thanked: 53 times
Been thanked: 1202 times

Re: pCompress bugs

Post by dimkr »

This person is responsible for many unexplained changes in woof-CE, like moving code from one directory to another, which breaks change history for files inside. This person also wrote big piles of code that clearly wasn't tested before it was merged, things like https://github.com/puppylinux-woof-CE/woof-CE/pull/3443. To make thing worse, there are some binaries in woof-CE, built by this person, and I've always wondered if any of them malicious and that's why they're not built from source by woof-CE.

Not knowing how to use git properly is one thing, but doing suspicious things and pushing untested code (with side effects like slowing down boot) is another.

You have very good reasons to read this person's code carefully and critically.

Many of the reasons why I forked woof-CE and started destroying and cleaning up things can be traced back to this person.

Post Reply

Return to “Bug Reports”