helpGNU Core Utilities - Support: sr #107466, compatibility patch


You are not allowed to post comments on this tracker with your current authentication level.

sr #107466: compatibility patch

Submitted by:  Adam Katz <khopesh>
Submitted on:  Fri 27 Aug 2010 08:25:31 PM UTC  
Category: NonePriority: 5 - Normal
Severity: 3 - NormalStatus: None
Privacy: PublicAssigned to: None
Open/Closed: OpenOperating System: *BSD

Fri 27 Aug 2010 08:25:31 PM UTC, original submission:

(I apparently can't create a bug or patch for this project, so I'm posting here instead.)

Aren't variables always supposed to be declared at the top of a function or the beginning of a scope area? I'm 6+ years rusty on C code, but compiling coreutils on FreeBSD 4.11 with GCC 2.95.4 produced a LOT of undefined variable errors.

Another issue is with verify(), which I assume declares variables in it as well and thus also needs to be at the top of a function or scope.

I've attached the modifications I needed to make it work (note, I obtained the source from Debian's coreutils-8.5-1 package, which might include some of their own patches).

Please pay extra attention to shred.c as I'm not sure if moving that verify call is safe. If not, some extra curly brackets on the original code should suffice.

Adam Katz <khopesh>


Attached Files
file #21325:  coreutils-8.5.patch added by khopesh (21KiB - text/x-patch - moved a bunch of declarations to tops of functions and scopes)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by khopesh (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Please enter the title of George Orwell's famous dystopian book (it's a date):



    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 27 Aug 2010 08:25:31 PM UTCkhopeshAttached File-=>Added coreutils-8.5.patch, #21325

    Back to the top

    Powered by Savane 3.1-cleanup1