Ticket #3089 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

Uses deprecated "find -perm +xxx" syntax, please switch to "find -perm /+++"

Reported by: onlyjob Owned by:
Priority: minor Milestone: 4.8.11
Component: mc-core Version: 4.8.10
Keywords: Cc:
Blocked By: Blocking:
Branch state: no branch Votes for changeset:

Description

As reported in http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=724766

the mc package uses find -perm +xxx. This GNU extension
syntax is deprecated and does not work in GNU findutils 4.5.11 and
later. Please change to find -perm /xxx instead.

The new syntax was introduced in 4.2.25 (October 2005) and should
therefore be available on any relevant system.

ametzler@argenau:/tmp/BUGS$ grep -lr -- '-perm +'
mc-4.8.10/src/filemanager/panelize.c

Change History

comment:1 Changed 4 years ago by andrew_b

  • Blocked By 3051 added
  • Milestone changed from Future Releases to 4.8.11

comment:2 Changed 4 years ago by egmont

A quick google search for "find" manuals suggest that non-GNU implementations probably don't understand the "/" syntax. So I think we'd need to autodetect (configure, or runtime).

comment:3 Changed 4 years ago by andrew_b

  • Blocked By 3051 removed

comment:4 Changed 4 years ago by andrew_b

  • Status changed from new to closed
  • Resolution set to fixed

Let's wait bugreport about unsupported syntax.

At current time, fixed as [50734c071674c83412dc42dca663534add6ae7d1].

Note: See TracTickets for help on using tickets.