Author Topic: PierreF's patchset  (Read 3435 times)

Greudin

  • Posts: 131
  • Turrets: +1/-1
    • http://tremulous.bricosoft.com
PierreF's patchset
« 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.


Developpers! Developpers! Clap! Clap!
url=http://tremulous.bricosoft.com][/url]

Caveman

  • Guest
PierreF's patchset
« Reply #1 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.

kevlarman

  • Posts: 2737
  • Turrets: +291/-295
PierreF's patchset
« Reply #2 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
Quote from: Asvarox link=topic=8622.msg169333#msg169333
Ok let's plan it out. Asva, you are nub, go sit on rets, I will build, you two go feed like hell, you go pwn their asses, and everyone else camp in the hallway, roger?
the dretch bites.
-----
|..d| #
|.@.-##
-----

PierreF

  • Posts: 11
  • Turrets: +0/-0
PierreF's patchset
« Reply #3 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 :))

khalsa

  • Administrator
  • Posts: 597
  • Turrets: +187/-132
    • http://www.mercenariesguild.net
PierreF's patchset
« Reply #4 on: March 21, 2007, 07:05:29 am »
May I just say:

Everyone should use the Patch tracker to avoid such problems.


Khalsa
}MG{ Mercenariesguild
ਮਨੁ ਜੀਤੇ ਜਗੁ ਜੀਤਿਆ