Author Topic: COMPILE_BAF_TO_BCS and friends marked as DEPRECATED  (Read 420 times)

Offline DavidW

  • Planewalker
  • *****
  • Posts: 275
COMPILE_BAF_TO_BCS and friends marked as DEPRECATED
« on: March 01, 2019, 03:22:52 PM »
I just noticed that the readme says that COMPILE_BAF_TO_BCS et al are 'deprecated'. Can you relax this to 'do not use without a real reason' or similar? There aren't many cases where you wouldn't use DECOMPILE_AND_PATCH, but there are some - e.g., you have a template BAF file and you want to hot-modify it and install multiple versions (since COMPILE doesn't allow you to have different source and target filenames).

Offline CamDawg

  • Infidel
  • Planewalker
  • *****
  • Posts: 832
  • Dreaming of a red Xmas
    • The Gibberlings Three
Re: COMPILE_BAF_TO_BCS and friends marked as DEPRECATED
« Reply #1 on: March 01, 2019, 06:15:29 PM »
Second. FWIW, COMPILE_BAF_TO_BCS remains the simplest way to compile an AI script direct to the /scripts folder.
The Gibberlings Three - Home of IE Mods

The BG2 Fixpack - All the fixes of Baldurdash, plus a few hundred more. Now available, with more fixes being added in every release.

Offline The Imp

  • Planewalker
  • *****
  • Posts: 280
  • Gender: Male
Re: COMPILE_BAF_TO_BCS and friends marked as DEPRECATED
« Reply #2 on: March 03, 2019, 12:18:05 PM »
Yes, all that.
But being marked deprecated is a little different thing that "marked for deletion", in the word itself, it's just "not recommended" usually for a reason, of it being part of old coding style.

.. and don't you need to then copy the .bcs file to the scripts folder as a .BS file...

Offline DavidW

  • Planewalker
  • *****
  • Posts: 275
Re: COMPILE_BAF_TO_BCS and friends marked as DEPRECATED
« Reply #3 on: March 04, 2019, 10:17:26 AM »
No, because COMPILE_BAF_TO_BCS doesn’t impose the .bcs suffix.

Offline Wisp

  • Moderator
  • Planewalker
  • *****
  • Posts: 1051
Re: COMPILE_BAF_TO_BCS and friends marked as DEPRECATED
« Reply #4 on: March 07, 2019, 09:09:13 AM »
Suppose I added an OUT option for COMPILE that defaults to current behaviour but allows you to specify target directory and target name: would that satisfy you instead? I'm not keen on this don't use this unless you really think you must thing, because it's nonsense.

Offline DavidW

  • Planewalker
  • *****
  • Posts: 275
Re: COMPILE_BAF_TO_BCS and friends marked as DEPRECATED
« Reply #5 on: March 07, 2019, 10:33:05 PM »
I think that's fine, given that COMPILE already permits patches.

But is there anything actually technically-problematic about COMPILE_BAF_TO_BCS? Assuming there's nothing technically wrong with it, can you just drop the 'deprecated' and just put a note in the readme saying that 99% of the time DECOMPILE_AND_PATCH is better?

Offline Wisp

  • Moderator
  • Planewalker
  • *****
  • Posts: 1051
Re: COMPILE_BAF_TO_BCS and friends marked as DEPRECATED
« Reply #6 on: March 17, 2019, 01:25:24 PM »
There's nothing problematic about them other than their error-prone nature.

But what's the motivation for not recommending against the use of these patches?

Offline Wisp

  • Moderator
  • Planewalker
  • *****
  • Posts: 1051
Re: COMPILE_BAF_TO_BCS and friends marked as DEPRECATED
« Reply #7 on: March 23, 2019, 06:35:55 AM »
Wording has been relaxed.

 

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.

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)?: