Opened 14 years ago

Closed 5 years ago

Last modified 4 years ago

#5431 closed bug (fixed)

diff with wrong filenames

Reported by: Adek336 Owned by: nobody
Priority: normal Milestone: R1/beta2
Component: Applications/Command Line Tools Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

diff existing-file existing-file works fine
diff non-existant non-existant should write an error message for each file and return non-zero, it doesn't
diff existing-file non-existant and diff non-existant existing-file produces the error message "bad file descriptor"

hrev35505

Attachments (1)

patch (440 bytes ) - added by stpere 13 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 by axeld, 14 years ago

Version: R1/alpha1R1/Development

That could be a positive error code problem.

by stpere, 13 years ago

Attachment: patch added

comment:2 by stpere, 13 years ago

patch: 01

comment:3 by stpere, 13 years ago

indeed, the attached patch seems to fix it. however, I hesitate to commit it because I'm afraid that NONEXISTENT (==-2) and UNOPENED (==-1) that are internal constants in diffutils might collide with valid errno.

comment:4 by nielx, 12 years ago

patch: 10

comment:5 by scottmc, 12 years ago

If/when this is fixed it should be done in such as way that it can be upstreamed in diffutils, so we don't have to keep repatching it with new diffutil releases.

comment:6 by pulkomandy, 9 years ago

The recipe for diffutils 3.3 in haikuports still has this problem. Maybe the POSIX error mapper could be used to "fix" the error codes.

comment:7 by waddlesplash, 5 years ago

Resolution: fixed
Status: newclosed

comment:8 by nielx, 4 years ago

Milestone: R1R1/beta2

Assign tickets with status=closed and resolution=fixed within the R1/beta2 development window to the R1/beta2 Milestone

Note: See TracTickets for help on using tickets.