Author Topic: The IF_EVAL bug has been solved maintaining backwards compatibility.  (Read 5127 times)

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
See the, um, topic title   :P

All diffs are available from http://bigg.spellholdstudios.net/weidu_diffs, and contain all my diffs from a previous thread, plus the NO_IF_EVAL_BUG tp2 action and some reformatting of a couple of install-time strings (namely, if the mod has more than 4 components, there is no ASK_EVERY_COMPONENT etc, the installer will now display the name of the tp2 file; moreover, you can now nstall one component, not only [Y]es it. This can make it easier to tailor eventual custom tra files (EG @-1006= "Install Component [") can now be expressed as @-1006 = "What do you want to do with component [", or similar.

If you want the compiled executable PM me (according to the GPL under which WeiDU is released, I can distribute it, provided I make it clear that the file was modified by me and list the date of each change, but before embarking in an 'official' role I prefer to bone up on Ocaml a bit more). Don't use this executable for distributing your mods just yet, though!
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #1 on: November 15, 2005, 10:54:08 AM »
Does this mean that Weimer is gone for good?
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #2 on: November 15, 2005, 11:07:14 AM »
I haven't talked to him yet (one of the main reasons why I didn't give a link to a version with these diffs) and I hope not, however I'm sure you noticed a rather... long hiatus.
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #3 on: November 15, 2005, 12:24:01 PM »
I haven't talked to him yet (one of the main reasons why I didn't give a link to a version with these diffs) and I hope not, however I'm sure you noticed a rather... long hiatus.
Yeah, I noticed, that's why I'm asking...
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

Offline Loriel

  • Planewalker
  • *****
  • Posts: 390
  • Gender: Male
    • Loriel's Downloads
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #4 on: November 15, 2005, 02:04:34 PM »
Regarding the long hiatus:  It was my impression that weimer has a once-a-year long hiatus because of other responsibilities and to regain some semblance of sanity. ::)  Is this hiatus different than previous years?

EDIT: spelling

Offline Andyr

  • Dance Commander
  • PPG
  • Planewalker
  • *****
  • Posts: 3178
  • Gender: Male
    • The Gibberlings Three IE mod community
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #5 on: November 15, 2005, 06:59:21 PM »
I don't know if he has finished his doctorate yet, but if not it is highly possible that his end-of-year (or end-of-doctorate) examinations and meetings could be going on around this time of year.
"We are the Gibberlings Three, as merry a band as you ever did see..." - Home of IE mods

<jcompton> Suggested plugs include "Click here so Compton doesn't ban me. http://www.pocketplane.net/ub"

I am unfortunately not often about these days so the best way to get hold of me is via email.

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
BACKUP directory now created if missing.
« Reply #6 on: November 16, 2005, 07:28:52 AM »
I don't know if he has finished his doctorate yet, but if not it is highly possible that his end-of-year (or end-of-doctorate) examinations and meetings could be going on around this time of year.
Well, this'd explain it all  :)
at least he has lots of diffs to apply when he gets back  :D

Also, new addition: the BACKUP directory is created if missing; thanks to this, MKDIR ~a/b/c/d~ is now a shorter equivalent to
MKDIR ~a~
MKDIR ~a/b~
MKDIR ~a/b/c~
MKDIR ~a/b/c/d~
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Grim Squeaker

  • Fallen
  • Planewalker
  • *****
  • Posts: 1019
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #7 on: November 16, 2005, 09:08:00 AM »
Regarding the long hiatus:  It was my impression that weimer has a once-a-year long hiatus because of other responsibilities and to regain some semblance of sanity. ::)  Is this hiatus different than previous years?

EDIT: spelling

As well as the point Andy made I'd imagine he's also gotta be busy working on WeiNGINE.
"You alone can make my song take flight..."

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
specify language and components to install from the CLI.
« Reply #8 on: November 22, 2005, 10:25:19 AM »
CLI options --language X, --force-install X, --force-uninstall X, --force-install-rest X Y..., --force-uninstall-rest X Y... are added, to allow easier making of scripts to replicate an install (soon to follow).

SET_THE_INTERACTIVE_VARIABLE tp2 action added; the tp2 variable %INTERACTIVE% allows to track wether the install session is interactive or not (to emulate AT_NOT_INTERACTIVE_EXIT or whatever).

diffs, as usual, are available from here: http://bigg.spellholdstudios.net/weidu_diffs.
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
REQUIRE_FILE ~data/25dialog.bif~ works on multiple installs.
« Reply #9 on: November 24, 2005, 07:24:29 AM »
- Bugfix to --force-install X resulting in infinite loop if the mod component fails to install.
- REQUIRE_FILE, FORBID_FILE and FILE_EXIST behave as before, except that, if they are fed a bif file, they search it inside the key (so that this works also with multiple installs).

Diffs are available from the same place as the previous posts  :)

May one (or more) Mac users contact me for running a compatibility test for this (rater tricky) diff?
« Last Edit: November 24, 2005, 07:28:05 AM by the bigg »
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #10 on: November 24, 2005, 07:34:20 AM »
The Clis look very handy. Are the changes being tested (I mean by others then yourself) and will you be releasing 186, or will you wait for Weimer?
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #11 on: November 24, 2005, 07:45:35 AM »
Heh, I know they're useful - I just built my game yesterday  ;D 

However, before releasing I'll ask Weimer (which I haven't done) and try to learn Ocaml a bit more  ;)

Also, they aren't tested with SUBCOMPONENTS... :/
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #12 on: November 24, 2005, 09:14:52 AM »
Also, they aren't tested with SUBCOMPONENTS... :/
BTW SUBCOMPONENTS work not so well with install all option, i.e. the first component is installed, can something be done about it?
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #13 on: November 24, 2005, 11:16:13 AM »
Also, they aren't tested with SUBCOMPONENTS... :/
BTW SUBCOMPONENTS work not so well with install all option, i.e. the first component is installed, can something be done about it?
Err, how would you propose to change --yes with subcomponents?

BTW, in refinements components 20 and 21 are both part of the same subcomponent group.
setup-refinements --force-install-rest 21 20
and
setup-refinements --force-install-rest 20 21
will install 20, not 21;
setup-refinements --force-install 10 --force-uninstall 10
is not defined (IE don't do that).


EDIT: Seeming unrelated: ADD_KIT/PRO/MUS has better detection routines (EG you can add swashtF if sswashtF is already present as a kit) and will set the variable even if the kit is already present (ADD_KIT ~TRUECLASS~ parameter_list will still set ~%TRUECLASS%~ to 0)
« Last Edit: November 24, 2005, 11:24:49 AM by the bigg »
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #14 on: November 24, 2005, 11:23:19 AM »
Also, they aren't tested with SUBCOMPONENTS... :/
BTW SUBCOMPONENTS work not so well with install all option, i.e. the first component is installed, can something be done about it?
Err, how would you propose to change --yes with subcomponents?

BTW, in refinements components 20 and 21 are both part of the same subcomponent group.
setup-refinements --force-install-rest 21 20
and
setup-refinements --force-install-rest 20 21
will install 20, not 21;
setup-refinements --force-install 10 --force-uninstall 10
is not defined (IE don't do that).
I rather meant the default option given to user "ask about each, install etc" if one has e.g. 20 components and only one with subcomponenst it's still necessery to use ASK_EVERY_COMPONENT lest should the player choose "install all" and get subcomponent one. I'd propose that the said install all ignored all components with subcomponents.

EDIT>typo... though it was a funny one and maybe should have been preserved...
« Last Edit: November 24, 2005, 11:28:56 AM by Borsook »
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #15 on: November 24, 2005, 11:27:33 AM »
Also, they aren't tested with SUBCOMPONENTS... :/
BTW SUBCOMPONENTS work not so well with install all option, i.e. the first component is installed, can something be done about it?
Err, how would you propose to change --yes with subcomponents?

BTW, in refinements components 20 and 21 are both part of the same subcomponent group.
setup-refinements --force-install-rest 21 20
and
setup-refinements --force-install-rest 20 21
will install 20, not 21;
setup-refinements --force-install 10 --force-uninstall 10
is not defined (IE don't do that).
I rather meant the default option given to user "ask about each, install etc" if one has e.g. 20 components and only one with subcomponenst it still necessery to use ASK_EVERY_COMPONENT lest should the player choose "install all" and get subcomponent one. I'd propose that the said install all ignored all compomponents with subcomponents.

I can't see wether this is possible or not and wether this is a good idea or not (for example in Refinements you get a subcomponent choice for the language for the armor revision part, and then install armor revisions, so you won't get updated descriptions this way).
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #16 on: November 24, 2005, 11:32:53 AM »
Also, they aren't tested with SUBCOMPONENTS... :/
BTW SUBCOMPONENTS work not so well with install all option, i.e. the first component is installed, can something be done about it?
Err, how would you propose to change --yes with subcomponents?

BTW, in refinements components 20 and 21 are both part of the same subcomponent group.
setup-refinements --force-install-rest 21 20
and
setup-refinements --force-install-rest 20 21
will install 20, not 21;
setup-refinements --force-install 10 --force-uninstall 10
is not defined (IE don't do that).
I rather meant the default option given to user "ask about each, install etc" if one has e.g. 20 components and only one with subcomponenst it still necessery to use ASK_EVERY_COMPONENT lest should the player choose "install all" and get subcomponent one. I'd propose that the said install all ignored all compomponents with subcomponents.

I can't see wether this is possible or not and wether this is a good idea or not (for example in Refinements you get a subcomponent choice for the language for the armor revision part, and then install armor revisions, so you won't get updated descriptions this way).
Maybe I wasn't clear, I meant that even if the user does not want to be asked about each com and install all he should still be asked about those with subcomponents. In current situation either modder uses ASK_EVERY_COMPONENT and user has to answer Yes 20 times or the modder doesn't use it, user chooses install all and never knows there was a choice to be made.
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #17 on: November 24, 2005, 11:37:02 AM »
Maybe I wasn't clear, I meant that even if the user does not want to be asked about each com and install all he should still be asked about those with subcomponents. In current situation either modder uses ASK_EVERY_COMPONENT and user has to answer Yes 20 times or the modder doesn't use it, user chooses install all and never knows there was a choice to be made.
Ah, OK - after I'm done ironing out a couple of bugs/features from ADD_KIT/MUS/PRO  :D
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #18 on: November 24, 2005, 11:39:49 AM »
Maybe I wasn't clear, I meant that even if the user does not want to be asked about each com and install all he should still be asked about those with subcomponents. In current situation either modder uses ASK_EVERY_COMPONENT and user has to answer Yes 20 times or the modder doesn't use it, user chooses install all and never knows there was a choice to be made.
Ah, OK - after I'm done ironing out a couple of bugs/features from ADD_KIT/MUS/PRO  :D
Great.
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #19 on: November 24, 2005, 12:02:22 PM »
Diffs (and executable for those who know the link) have just been updated with the aforementioned ADD_KIT fixes (I didn't touch the discussed issue with subcomponents).
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #20 on: November 24, 2005, 12:05:43 PM »
Diffs (and executable for those who know the link) have just been updated with the aforementioned ADD_KIT fixes (I didn't touch the discussed issue with subcomponents).
You work fast! Hmm and where pray is the thumbs up icon? Oh, well, think back to PnP and visualize ;D
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #21 on: November 24, 2005, 12:35:18 PM »
Nah, this was all the diff (as in diff -p -b -E):

Code: [Select]
*** 2837,2849 ****
          (PE_And(
          (Pred_File_Exists(PE_LiteralString "override/kitlist.2da")),
          (Pred_File_Contains(PE_LiteralString "override/kitlist.2da",
!           PE_LiteralString k.kit_name))))) then begin
          log_and_print "\n\nERROR: Kit [%s] already present! Skipping!\n\n"
          k.kit_name
        end else begin
--- 2904,2916 ----
          (PE_And(
          (Pred_File_Exists(PE_LiteralString "override/kitlist.2da")),
          (Pred_File_Contains(PE_LiteralString "override/kitlist.2da",
!           PE_LiteralString ("^" ^ k.kit_name ^ "\\b")))))) then begin
!         Var.set_int32 (k.kit_name) (Bcs.int_of_sym game "KITLIST.2DA" k.kit_name) ;
          log_and_print "\n\nERROR: Kit [%s] already present! Skipping!\n\n"
          k.kit_name
        end else begin
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #22 on: November 25, 2005, 07:59:42 AM »
Slight fix to the cli options. Updated fix and executable.
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline the bigg

  • The Avatar of Fighter / Thieves
  • Moderator
  • Planewalker
  • *****
  • Posts: 3804
  • Gender: Male
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #23 on: November 25, 2005, 12:30:56 PM »
Maybe I wasn't clear, I meant that even if the user does not want to be asked about each com and install all he should still be asked about those with subcomponents. In current situation either modder uses ASK_EVERY_COMPONENT and user has to answer Yes 20 times or the modder doesn't use it, user chooses install all and never knows there was a choice to be made.
...done  :)
Author or Co-Author: WeiDU (http://j.mp/bLtjOn) - Widescreen (http://j.mp/aKAiqG) - Generalized Biffing (http://j.mp/aVgw3U) - Refinements (http://j.mp/bLHoCc) - TB#Tweaks (http://j.mp/ba02Eg) - IWD2Tweaks (http://j.mp/98OFYY) - TB#Characters (http://j.mp/ak8J55) - Traify Tool (http://j.mp/g1Ry9A) - Some mods that I won't mention in public
Maintainer: Semi-Multi Clerics (http://j.mp/9UeIwB) - Nalia Mod (http://j.mp/dng9l0) - Nvidia Fix (http://j.mp/aRWjjg)
Code dumps: Detect custom secondary types (http://j.mp/hVzzXG) - Stutter Investigator (http://j.mp/gdtBn8)

If possible, send diffs, translations and other contributions using Git (http://j.mp/aBZFrq).

Offline Borsook

  • off topic
  • Planewalker
  • *****
  • Posts: 1679
  • Gender: Male
  • not a badger
Re: The IF_EVAL bug has been solved maintaining backwards compatibility.
« Reply #24 on: November 25, 2005, 12:35:13 PM »
Maybe I wasn't clear, I meant that even if the user does not want to be asked about each com and install all he should still be asked about those with subcomponents. In current situation either modder uses ASK_EVERY_COMPONENT and user has to answer Yes 20 times or the modder doesn't use it, user chooses install all and never knows there was a choice to be made.
...done  :)
Wow. Can I use it for distribution, can I? can I?
"Borsook seems like a pretty cool guy who offers his insight where appropriate... just very, very frequently." - Imrahil

Borsook's Chambers - Home of Borsook's Mods

 

With Quick-Reply you can write a post when viewing a topic without loading a new page. You can still use bulletin board code and smileys as you would in a normal post.

Warning: this topic has not been posted in for at least 120 days.
Unless you're sure you want to reply, please consider starting a new topic.

Name: Email:
Verification:
Type the letters shown in the picture
Listen to the letters / Request another image
Type the letters shown in the picture:
What color is grass?:
What is the seventh word in this sentence?:
What is five minus two (use the full word)?: