patchGNUstep - Patches: patch #8061, apple-gnu-gnu on OSX and other

Group
 
 

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

patch #8061: apple-gnu-gnu on OSX and other

Submitter:  Marat Ibadinov <ibadinov>
Submitted:  Wed 29 May 2013 01:45:51 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open

Wed 29 May 2013 02:39:44 PM UTC, comment #1: 

I think we really need patches to be done as distinct managable patch files, plus an explanation of what they are for, plus testcases to demonstrate what they do.

Having a look at the zip file for the things that I'm most involved with ...

sqlcient ... at first glance this patch just reverts code to an older version and is wrong, but perhaps I'm missing something.  If I'm not, then this patch can just be removed.

make ... this looks like three parts:
1. a change to Testing.h which is just wrong/unnecessary
2. huge change to 'configure', which shouldn't be in a patch since it's regenerated automatically
3. change to configure.ac to change the default library combo on apple systems ... but why?  Maybe there's an argument for it, but presumably it would be a nasty surprise for people using the existing code.
Maybe this patch should just be removed, or maybe there's good reason for the last part.

base ...  too big to give a quick response to.  this needs to be broken down into sections.  Some big things I see:
Changing to use symbolic constants consistently in version macros ... I like.
Large changes which just make indentation inconsistent with the coding standards ... we need to remove that sort of thing from any patch.
But in general, other changes can't be evaluated quickly ... there seem to be a lot of unsigned->NSUInteger changes, some of which appear to be in non-apple APIs which shouldn't be changed this way, but some of which probably do need to be changed.  We need to check which is which on an individual basis.

Richard Frith-Macdonald <CaS>
Group Member
Wed 29 May 2013 01:45:51 PM UTC, original submission:  

Patches announced on gnustep-dev mailing list under the subject “GNUstep on OSX”

Marat Ibadinov <ibadinov>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #28202:  patches.zip added by ibadinov (2MiB - application/zip)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by CaS (Posted a comment)
  • -email is unavailable- added by ibadinov (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-05-29 ibadinov Attached File- Added patches.zip, #28202

    Back to the top

    Powered by Savane 3.13-8ccc.
    Corresponding source code