Patch: Build UNDER Slopes

Forum for technical discussions regarding development. If you have a general suggestion, problem or comment, please use one of the other forums.

Moderator: OpenTTD Developers

How?

Cool :)
112
96%
Crap :)
5
4%
 
Total votes: 117

Ev
Engineer
Engineer
Posts: 21
Joined: 19 Jun 2007 18:59

Post by Ev »

----------
Last edited by Ev on 27 Jun 2007 22:01, edited 1 time in total.
peter1138
OpenTTD Developer
OpenTTD Developer
Posts: 1732
Joined: 30 Mar 2005 09:43

Post by peter1138 »

This patch will not be accepted in its current form with macros scattered over it. As the author has refused to clear this up, this patch will remain out of trunk, unless someone else decides to fix it.
He's like, some kind of OpenTTD developer.
Ev
Engineer
Engineer
Posts: 21
Joined: 19 Jun 2007 18:59

Post by Ev »

As the ignorance may not dictate other people what to do, this patch "will remain out of trunk" in any form.
Don't modify or include it in ottd, remove copies.
I had runned into developer's idiocity and ignorance so much times that it leave no desire to code and play ottd.
Fare you well.
Last edited by Ev on 27 Jun 2007 22:17, edited 1 time in total.
kaan
Route Supervisor
Route Supervisor
Posts: 399
Joined: 02 Apr 2007 20:13
Location: Nørup, Denmark

Post by kaan »

Well then ...
Can someone repost the patch please, I never did get a look at it :)

Code: Select all

if (YouAreHappyAndYouKnowIt) {
    ClapYourHands();
}
User avatar
NukeBuster
Traffic Manager
Traffic Manager
Posts: 222
Joined: 04 Jan 2006 18:16
Location: Alphen aan den Rijn, The Netherlands
Contact:

Post by NukeBuster »

kaan wrote:Well then ...
Can someone repost the patch please, I never did get a look at it :)
I was just having the same thought... too bad I never saved it.
NukeBuster

Transport Empire: The Transport Empire Linux effort
Join the Transport Empire IRC channel: [url]irc://irc.oftc.net/transportempire[/url] !

OpenTTD patch(es): Password at join
MJS
Director
Director
Posts: 540
Joined: 28 Jul 2005 09:31

Post by MJS »

Ev wrote:Don't modify or include it in ottd, remove copies.
So I'll put my copy online just so you can have a look at it. I have no clue about Evs coding style, which apparently put him into trouble; but I believe we should at least pay him the respect of not using his code. Still, if anyone wants to take a peak at the code to get ideas for a similar patch...

If Ev disagrees with me putting a copy back up, then I wouldn't complain if any mod would be so kind as to remove the attachment from this post again.

PS This is not the latest version, unfortunately; this is before Ev made some fixes to bring the code slightly more in line with general coding styles.
Attachments
build_under_slopesv02_181.patch
(12.94 KiB) Downloaded 167 times
kaan
Route Supervisor
Route Supervisor
Posts: 399
Joined: 02 Apr 2007 20:13
Location: Nørup, Denmark

Post by kaan »

MJS wrote:
Ev wrote:Don't modify or include it in ottd, remove copies.
So I'll put my copy online just so you can have a look at it. I have no clue about Evs coding style, which apparently put him into trouble; but I believe we should at least pay him the respect of not using his code. Still, if anyone wants to take a peak at the code to get ideas for a similar patch...

If Ev disagrees with me putting a copy back up, then I wouldn't complain if any mod would be so kind as to remove the attachment from this post again.

PS This is not the latest version, unfortunately; this is before Ev made some fixes to bring the code slightly more in line with general coding styles.
Thanks :)

Ev may or may not have the right to decide what happens to this patch.
In my opinion this patch is under the GPL simply because its a derived work of OTTD. If that is the case then it is open source and we can keep it and include any part of it we wish.

Code: Select all

if (YouAreHappyAndYouKnowIt) {
    ClapYourHands();
}
User avatar
Digitalfox
Chief Executive
Chief Executive
Posts: 708
Joined: 28 Oct 2004 04:42
Location: Catch the Fox if you can... Almost 20 years and counting!

Post by Digitalfox »

Ev wrote:As the ignorance may not dictate other people what to do, this patch "will remain out of trunk" in any form.
Don't modify or include it in ottd, remove copies.
I had runned into developer's idiocity and ignorance so much times that it leave no desire to code and play ottd.
Fare you well.
I don't know what happen with you and dev's..

But we users don't have any guilty, and by removing your work, you're just looking like are hating everybody.. :roll:
Sacro
Tycoon
Tycoon
Posts: 1145
Joined: 18 Jun 2005 21:08
Location: Here
Contact:

Post by Sacro »

Here is (AFAIK) the latest version, snagged yesterday, i was planning on seeing how much work was needed to bring it into line.
Attachments
build_under_slopesv061r10351_173.patch
(30.18 KiB) Downloaded 205 times
We Am De Best

Host of ThroughTheTube site
User avatar
Ailure
Route Supervisor
Route Supervisor
Posts: 435
Joined: 26 Apr 2005 19:06
Location: Sweden

Post by Ailure »

Indeed, as someone pointed out on IRC, according to the GPL:
b) You must cause any work that you distribute or publish, that in whole or in part contains or is derived from the Program or any part thereof, to be licensed as a whole at no charge to all third parties under the terms of this License.
So it's fully legal to continue work on his patch, whenever he wants that is another thing.
User avatar
athanasios
Tycoon
Tycoon
Posts: 3138
Joined: 23 Jun 2005 00:09
Contact:

Post by athanasios »

I DON'T THINK SO. :evil:
EV was clear and sharp. So some respect to him. He never said that his patch was GPLd. If someone wants to continue with his patch he must PM to him and ask for his licence. And his statement that he will never play OTTD again was made out of anger; leave him some time to cool down. We don't have the luxury to miss another valuable member of the forums.
http://members.fortunecity.com/gamesart
"If no one is a fool I am also a fool." -The TTD maniac.


I prefer to be contacted through PMs. Thanks.
User avatar
Ailure
Route Supervisor
Route Supervisor
Posts: 435
Joined: 26 Apr 2005 19:06
Location: Sweden

Post by Ailure »

Technically, from a legal standpoint his patch is under the GPL as it's "derived" from a codebase that is... GPL. Unless he wrote the patch manually without even looking at the openTTD sourcecode, as far I know it's GPL.

IANAL though.
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

I'm with Ailure. A patch is a derived work of the code on which it is based. If that is the case (IANAL) the patch, being a distributed derived work of a GPLed work, must also be GPL. The fact that it was removed does not un-GPL it, nor does/will/can any other action that Ev may or may not take. Permissions granted under the GPL cannot be rescinded.

Therefore, Sacro is quite within his rights to redistribute the patch, since he received it legally[0], and GPL permits verbatim redistribution.


[0] This is important! Having your code stolen does not count as distribution, and so the distribution provisions of the GPL do not apply in that case. The GNU website, in fact, covers this case.
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
Haukinger
Engineer
Engineer
Posts: 110
Joined: 15 Mar 2006 16:38

Post by Haukinger »

DaleStan wrote:If that is the case (IANAL) the patch, being a distributed derived work of a GPLed work, must also be GPL.
You say it _must_ be GPL, but that doesn't mean it actually is. You might force the author with a lawsuit to release it under GPL, but just because it has to be released under GPL, you can't possibly ignore the actual license. Afterall, the whole ottd project is kind of pirated, then how can something derived from it be legally GPL ?
I fully understand Ev.
DeletedUser21
Tycoon
Tycoon
Posts: 11501
Joined: 20 Sep 2004 22:45

Post by DeletedUser21 »

Whoa! Recap, Ev made a patch, but got rejected because of the coding style, now Ev leaves in anger. Now I don't know what happend but it sounds a bit lame to me, if it's broken why not fix it? If the code is to dirty for inclusion, then unravel it. I mean why all the extra hassle?
DaleStan
TTDPatch Developer
TTDPatch Developer
Posts: 10285
Joined: 18 Feb 2004 03:06
Contact:

Post by DaleStan »

Haukinger:
The GPL cannot be used to force anyone to distribute anything unless binaries have been provided. But that's immaterial. The patch has already been released. The discussion now is whether the GPL covers its release.

I believe you're trying to arguing that OpenTTD is not technically GPLed? The way I see it, you have two choices for that argument:
1) The application of the GPL to OpenTTD is not legal.
2) The application of the GPL to OpenTTD is legal.

If the first, you are yourself guilty of copyright violation, because you have a copy of OTTD. Unless you have some other license from some party who *is* authorized to license the OpenTTD code, you're producing some pretty interesting self-incriminating testimony here.
If the second, then OpenTTD is legally GPL, and Ev's patch is therefore required to be GPL.

Mr. X:
Because Ev is an otherwise intelligent coder who can't admit that he could have possibly been wrong. (Something like an extreme version of me, come to think of it.)
To get a good answer, ask a Smart Question. Similarly, if you want a bug fixed, write a Useful Bug Report. No TTDPatch crashlog? Then follow directions.
Projects: NFORenum (download) | PlaneSet (Website) | grfcodec (download) | grfdebug.log parser
User avatar
mamuf
Engineer
Engineer
Posts: 33
Joined: 30 Mar 2007 21:31
Location: Prague, CZ

Post by mamuf »

I don't know where is the truth - can we continue to work on the original code of this patch or can not we?

But as I am looking at the code itself, these macros could be easilly converted into functions and I really don't see any reason why these pieces of code are macros. The functions should be defined inline and then we can ask for merge with the trunk hapilly, I think :-).
User avatar
prissi
Chief Executive
Chief Executive
Posts: 647
Joined: 15 Nov 2004 19:46
Location: Berlin, Germany
Contact:

Post by prissi »

Technically, if you have published something under GPL, it remains GPLed. If the patch was GPL (but whithout a statement one really does not know), then it would remain GPL. The patch is not a derived work, since it was not distributed in binary form and without the original source.

I am entitled to modify any GPL program for my personal use without publishing my changes if I do not distribute in any form. Doing a patch is imho not distribution. Therefore, imho it is legal (although not nice) to withdraw a patch. Anyway, there will be lots of bitrot fast.

And looking at the latest versions, macros defining that lives only some lines is not a good programming style and obfuscating things a lot.
Haukinger
Engineer
Engineer
Posts: 110
Joined: 15 Mar 2006 16:38

Post by Haukinger »

DaleStan wrote:Haukinger:If the second, then OpenTTD is legally GPL, and Ev's patch is therefore required to be GPL.
Exactly. "Required to be GPL". Not "released under GPL". That is, you can force him to release it under GPL. Until that is done, he can release it under whatever license he wants.

My point was that some one who distributes and develops an essentially pirated game shouldn't be so narrow-minded when it comes to licensing.
MJS
Director
Director
Posts: 540
Joined: 28 Jul 2005 09:31

Post by MJS »

So shan't we just agree that we do not use Ev's code, but that someone writes a similar patch of his own? Whether or not he draws his inspiration from Ev's code is then wholly his own choice, which we need not know about.
Locked

Return to “OpenTTD Development”

Who is online

Users browsing this forum: No registered users and 47 guests