Page 1 of 1

pFind bug

Posted: Wed Oct 09, 2024 8:22 am
by MochiMoppel

Observed in pFind 6.3 of BookwormPup64 10.0.8

pFind never finds any match when selecting
[X] Search only current filesystem (IMHO this should be the default!)
Permissions in the "Mode" category. e.g. the very typical Read access for all and Write access for user

When constructing the find command pFind fails to separate the -perm 644 and -xdev arguments with the mandatory whitespace, resulting in an invalid command construct (see statusbar).

I don't expect that this will be fixed. pFind is unmaintained for a long time and has quite a few unresolved problems. Just stating it here for the record.


Re: pFind bug

Posted: Wed Oct 09, 2024 10:37 am
by rockedge

time to look at the code and see it is fixed for the future.


Re: pFind bug

Posted: Wed Oct 09, 2024 1:22 pm
by wizard

@MochiMoppel
@rockedge

Cannot confirm bug based on following test.

BookwormPup64 10.0.7 and 10.0.8.
Pfind 6.3
search for = smb
Where to search = System files
Search only current file system

Returns expected results.

Did I miss something?

Thanks
wizard


Re: pFind bug

Posted: Wed Oct 09, 2024 1:34 pm
by bigpup

@MochiMoppel

you seem to have a good understanding of the code in this program.

If no one is maintaining it.

How about you taking it over and do the maintaining?

This is one of the must have programs in any Puppy Linux version.

sure would be good if it stayed working correctly.


Contact Zigbert

Posted: Wed Oct 09, 2024 4:43 pm
by don570

Zigbert will solve the problem if he is asked.

I pointed out a problem with Dollar signs and other strange characters in folder names
and he solved it
Remember that the name of the program is pfind/pfilesearch when contacting Zigbert
https://forum.puppylinux.com/viewtopic.php?t=6672
________________________________________________________-

Screenshot 2024-10-09 12.44.25.png
Screenshot 2024-10-09 12.44.25.png (15.43 KiB) Viewed 441 times

Re: pFind bug

Posted: Thu Oct 10, 2024 1:55 am
by MochiMoppel
wizard wrote: Wed Oct 09, 2024 1:22 pm

Returns expected results.
Did I miss something?

Permissions :mrgreen:


Re: pFind bug

Posted: Thu Oct 10, 2024 6:00 am
by peebee

Please provide the fix and I will apply it to:
https://github.com/puppylinux-woof-CE/w ... ages/pfind

for future Woof-CE builds.


Re: pFind bug

Posted: Fri Oct 11, 2024 2:18 am
by MochiMoppel

pFind will also fail when using these options:
☐ Search also subdirectories
☒ Search only current filesystem

This all points to an obvious cause.. and raises a suspicion that some options in pFind are never used 😧


Re: pFind bug

Posted: Fri Oct 11, 2024 7:09 am
by Dry Falls

Using pfind-6.2-2-x86_64 in lighthouse (slackware64-15.0 and current), all of the bugs I experienced (the above and more) disappeared when I rolled back gtkdialog from gtkdialog-0.8.5 to gtkdialog-0.8.3 which I just compiled from slackbuilds. As well, gtkdialog-0.8.3-x86_64-1_SBo no longer requires mesa (opengl and vulkan-sdk) and llvm libraries to be present to avoid crashing. (These are humongous libraries ).

This might help.
df


Re: pFind bug

Posted: Fri Oct 11, 2024 11:27 am
by MochiMoppel

@peebee Add the missing space to line 76 of
https://github.com/puppylinux-woof-CE/w ... earch/func

[ "$CHKXDEV" = "true" ] && XDEV=" -xdev" || XDEV=""

And just in case you are wondering: Yes, this bug exists since the very first version.
Found this in pfilesearch v.1.11 of 2008:
if [ $CHKXDEV = "true" ]; then XDEV="-xdev"; else XDEV=""; fi :shock:

@Dry Falls I doubt that this has anything to do with gtkdialog. Which other bugs did you experience?


Re: pFind bug

Posted: Fri Oct 11, 2024 1:07 pm
by peebee

Re: pFind bug

Posted: Fri Oct 11, 2024 3:31 pm
by fredx181
MochiMoppel wrote: Fri Oct 11, 2024 11:27 am

...
And just in case you are wondering: Yes, this bug exists since the very first version.
Found this in pfilesearch v.1.11 of 2008:
if [ $CHKXDEV = "true" ]; then XDEV="-xdev"; else XDEV=""; fi :shock:

Yes, you are very, very late discovering this bug ! :lol:

Works fine now with the space before -xdev XDEV=" -xdev", thanks :thumbup: