Tremulous Forum

Community => Servers => Topic started by: Greudin on March 12, 2007, 10:58:06 pm

Title: PierreF's patchset
Post by: Greudin on March 12, 2007, 10:58:06 pm
This is a set of patches (thank you all developpers !) arranged by PierreF's

Download :
http://trem.yi.org/tremwiki/Download

The list of patches :
http://trem.yi.org/tremwiki/Modification

It is currently used by the some eu servers : tHc, Amphetamines, Bricosoft, Excessive.

(http://greudin.free.fr/medias/image/Developers.gif)
Developpers! Developpers! Clap! Clap!
Title: PierreF's patchset
Post by: Caveman on March 13, 2007, 01:45:13 am
Great, another place that mirrors patches.

That said...

But the reason why this site exists is beyond me.

As for Pierre's genuine patches, they are available only via mail.

Sure he he posts the SVN he is actually using and giving a link to his source, but fails to mention what SVN-version he did use.
So the making of a diff to generate a patch is highly unlikly to succeed.

It is also quite nice that he gives proper credits to DASPRID, but why does the link to his patch not lead to the patch itself, as it should when making the words "LogTeamClass.patch" a link?

The same with Risujin's one-liner. Why is it omitted?
Because it's just one line?
Sorry but here is also a link missing and mandatory imho.

While the link to the AW-Patches leads to a selection against a number of SVN-Versions, it is kinda misleading then to list other devs patches under the same heading and thus making it look like they are all from us.


All in all i'd say this site should have stayed offline as it is not ready yet.
Title: PierreF's patchset
Post by: kevlarman on March 13, 2007, 04:09:31 am
Quote from: "Caveman"

It is also quite nice that he gives proper credits to DASPRID, but why does the link to his patch not lead to the patch itself, as it should when making the words "LogTeamClass.patch" a link?
actually wolfwings wrote the patch, not DASPRiD
Title: PierreF's patchset
Post by: PierreF on March 13, 2007, 01:28:16 pm
Quote from: "Caveman"
Great, another place that mirrors patches.

That said...

But the reason why this site exists is beyond me.


It's mostly for people who wants to use my bundle of patch (mostly people i know from IRC)

Quote from: "Caveman"

As for Pierre's genuine patches, they are available only via mail.


Ok, I will make the .patch. I was too lazy to make them :)

Quote from: "Caveman"


Sure he he posts the SVN he is actually using and giving a link to his source, but fails to mention what SVN-version he did use.

It's writen on HomePage it's svn 909. I added this also on Modification page.

Quote from: "Caveman"

It is also quite nice that he gives proper credits to DASPRID, but why does the link to his patch not lead to the patch itself, as it should when making the words "LogTeamClass.patch" a link?

I don't make LogTeamClass a link... it's the wiki engine which make link any word with capital in the word. I will fix it.

Quote from: "Caveman"

The same with Risujin's one-liner. Why is it omitted?
Because it's just one line?
Sorry but here is also a link missing and mandatory imho.


I will fix this

Quote from: "Caveman"

While the link to the AW-Patches leads to a selection against a number of SVN-Versions, it is kinda misleading then to list other devs patches under the same heading and thus making it look like they are all from us.


The link leads to a selection against a number of svn-versions because Wraths patches didn't exist for svn-909 which I use.
I added note to remind that Wraths patches are a bundle of patches from other developers.


Thank for your commentaries, they will improve this wiki. (and it's a wiki, you can correct it if you see a mistake :))
Title: PierreF's patchset
Post by: khalsa on March 21, 2007, 07:05:29 am
May I just say:

Everyone should use the Patch (http://www.mercenariesguild.net/patches/) tracker (http://tremulous.net/phpBB2/viewtopic.php?t=3926) to avoid such problems.


Khalsa