Jump to content
Eternal Lands Official Forums
Wytter

-dnew_structure

Recommended Posts

Hey all

 

I spoke with Entropy about the implementation of the native language support and he suggested that the following structure was to be used:

 

./languages/en/<subfolders (optional)>/<files>

i.e.:

./languages/en/Encyclopedia/index.xml

 

After merging this structure in my eldir find reports the following:

languages/
languages/en
languages/en/knowledge.lst
languages/en/strings
languages/en/strings/options.xml
languages/en/strings/console.xml
languages/en/strings/errors.xml
languages/en/strings/spells.xml
languages/en/strings/stats.xml
languages/en/strings/help.xml
languages/en/Encyclopedia
languages/en/Encyclopedia/Potions.xml
languages/en/Encyclopedia/Weapons.xml
languages/en/Encyclopedia/Magic.xml
languages/en/Encyclopedia/Jewelry.xml
languages/en/Encyclopedia/Jewlery.xml
languages/en/Encyclopedia/General.xml
languages/en/Encyclopedia/Clothes.xml
languages/en/Encyclopedia/Books.xml
languages/en/Encyclopedia/images
languages/en/Encyclopedia/images/monsters1.bmp
languages/en/Encyclopedia/images/monsters2.bmp
languages/en/Encyclopedia/images/monsters3.bmp
languages/en/Encyclopedia/Alchemy.xml
languages/en/Encyclopedia/index.xml
languages/en/Encyclopedia/Monsters.xml

If it fails to load an entity in your specified language (#language=en), then it'll fall back on english.

 

In order to still support the old structure, I added the target -DNEW_STRUCTURE that you'll have to add to your Makefile. The NEW_STRUCTURE will not work with the old folder structure, so be sure that you've set the target right.

 

Furthermore I added -DLOAD_XML to the default linux makefile.

Share this post


Link to post
Share on other sites

It still looks for the images in Encyclopedia/images.. Which is as it should be, I think, as we won't have different images for different languages.

So I have:

Encyclopedia/images/monsters1.bmp (etc.)
languages/en/Encyclopedia/ (all the english xmls)

 

(I can't test if this actually works in game at the moment, but at least I get no error messages :angry:)

Share this post


Link to post
Share on other sites

Just an small thing about the paths. You have been writting them in these 2 ways:

"./languages/%s/strings/%s"

"languages/%s/strings/%s" -> Can you use this format instead of the first one, it can give problems in windows, files are not found so it crashes on startup.

Share this post


Link to post
Share on other sites
Just an small thing about the paths. You have been writting them in these 2 ways:

"./languages/%s/strings/%s"

"languages/%s/strings/%s"  -> Can you use this format instead of the first one, it can give problems in windows, files are not found so it crashes on startup.

Of course, sorry for the inconvinience :-)

 

Perhaps the images for the encyclopedia should be in textures/Encyclopedia/? Then the text and images would be located in different places. The reason why it loads them from Encyclopedia is because the path to the image is in xml. If the placement changes the xml-files paths will have to change as well.

Share this post


Link to post
Share on other sites

We have too many defines in the client, it's bulky. Let's get rid of the 'old way' and make the dnew_structure code default (and also get rid of that define),

Share this post


Link to post
Share on other sites

A lot of defines that we have in there should have only lasted until the code that they surrounded became stable. Vertex arrays, cache system, and music defines should all be removed, IMO.

Share this post


Link to post
Share on other sites

Yeah, remove them, but let the vertex arrays there, we should let Learner remove it. Besides, after we switch to Cal3D, we won't need it anyway.

Share this post


Link to post
Share on other sites

I noticed the research window didn't work anymore... Turned out I had forgotten to move knowledge.lst to languages/en. The weird thing is that I got no warnings or errors of any kind, just an empty research window. <_<

Perhaps the client should warn if this file is missing...

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

  • Recently Browsing   0 members

    No registered users viewing this page.

×