Re: [warzone2100-dev] Directory layout changes for future build system
On Fri, Jan 28, 2011 at 2:10 AM, Safety0ff safety0ff@gmail.com wrote: Next subject: Opinions on moving miniupnpc from lib/netplay to somewhere else? I don't really care either way. If you feel like moving it, I'm fine with that. - Per ___ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev
Re: [warzone2100-dev] Directory layout changes for future build system
It seems there isn't enough support for this change (moving lib/* to src/). Next subject: Opinions on moving miniupnpc from lib/netplay to somewhere else? ___ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev
Re: [warzone2100-dev] Directory layout changes for future build system
On Sat, Jan 15, 2011 at 7:18 AM, Safety0ff safety0ff@gmail.com wrote: While working on a CMake build system I took the opportunity to move the contents from lib to src. Generally, I am worried that we are now making so many huge changes to the codebase that all existing patches soon have to be written from scratch. This is not cool. Specifically, I do not see a good reason for this change. Compiling parts of a codebase to a temporary library is quite normal practice, and I like having the code available in a shallow structure. - Per ___ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev
Re: [warzone2100-dev] Directory layout changes for future build system
On Saturday, 15 January 2011 at 1:18, Safety0ff wrote: Hi all, While working on a CMake build system I took the opportunity to move the contents from lib to src. I don't see what that gains us, to me it'll only lead to confusion between old and new layout and breaking of existing patches/branches. Somewhat loosely related, what would be good imo is a lib/3rdparty directory, for all the third party code we include. The rationale for this was: 1) It helped me to keep the build system rational without compiling some of the parts into libraries (like the current system does.) Even automake can build the whole Warzone without building separate libraries (though it's not configured that way atm). ___ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev
Re: [warzone2100-dev] Directory layout changes for future build system
On Jan 15, 2011, at 4:01 AM, Per Inge Mathisen wrote: On Sat, Jan 15, 2011 at 7:18 AM, Safety0ff safety0ff@gmail.com wrote: While working on a CMake build system I took the opportunity to move the contents from lib to src. I have no objection to the reorganising of the code base; in fact I am in favor of it. Generally, I am worried that we are now making so many huge changes to the codebase that all existing patches soon have to be written from scratch. Generally this is why I am in favor of using feature branches in personal forks instead of patches; no mater the changes made they will still be able to be rebased onto the apropreate branch (or merged if there has been a significant change of contents). -- My Web Sites: http://dak180.users.sourceforge.net/ smime.p7s Description: S/MIME cryptographic signature ___ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev
Re: [warzone2100-dev] Directory layout changes for future build system
On 11-01-15 04:01 AM, Per Inge Mathisen wrote: Generally, I am worried that we are now making so many huge changes to the codebase that all existing patches soon have to be written from scratch. This is not cool. Specifically, I do not see a good reason for this change. Compiling parts of a codebase to a temporary library is quite normal practice, and I like having the code available in a shallow structure. - Per I don't see how this is a bigger deal for patches than the c - cpp rename, it is more or less a rename plus a removal of the useless lib/ in the include directives, which shouldn't require rewriting a patch from scratch. Moving stuff from lib to src doesn't create a deeper structure, lib and src have the same number of characters, so files will be at the same depth no matter how you spin it. If you're alluding to me moving the contents of src to src/warzone2100 (in the CMake branch) then let me make it clear that the purpose of this thread of discussion was for the lib to src move *only*. I'm rather indifferent about that change. On 11-01-15 10:14 AM, Christian Ohm wrote: On Saturday, 15 January 2011 at 1:18, Safety0ff wrote: Hi all, While working on a CMake build system I took the opportunity to move the contents from lib to src. I don't see what that gains us, to me it'll only lead to confusion between old and new layout and breaking of existing patches/branches. Somewhat loosely related, what would be good imo is a lib/3rdparty directory, for all the third party code we include. The rationale for this was: 1) It helped me to keep the build system rational without compiling some of the parts into libraries (like the current system does.) Even automake can build the whole Warzone without building separate libraries (though it's not configured that way atm). I put third party code in a top level 3rdparty directory in the CMake branch, I didn't see a reason to put third party stuff in lib/3rdparty/*, that just seems unnecessary hierarchy. What I said does not imply that it was mandatory to build Warzone without building separate libraries, what was meant is that doing so kept things cleaner / more logical. I am aware that autotools can do that too, but IIRC you lose certain options (using separate build flags, IIRC). On 11-01-15 10:32 AM, dak180 wrote: Generally this is why I am in favor of using feature branches in personal forks instead of patches; no mater the changes made they will still be able to be rebased onto the apropreate branch (or merged if there has been a significant change of contents). Or you can use this if you have git patches: http://developer.wz2100.net/wiki/GitTricks#Merginganoldpatch ___ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev
[warzone2100-dev] Directory layout changes for future build system
Hi all, While working on a CMake build system I took the opportunity to move the contents from lib to src. The rationale for this was: 1) It helped me to keep the build system rational without compiling some of the parts into libraries (like the current system does.) 2) lib is more commonly used for: - The binary output directory for libraries. - A directory that contains pre-compiled dependencies. - A directory where dependencies can be placed for linking (for this sometimes the plural is used: libs) I would like this to be discussed so that we reach definite decision on the matter. Regards, -Safety0ff ___ Warzone-dev mailing list Warzone-dev@gna.org https://mail.gna.org/listinfo/warzone-dev