Rockbox

Sync’d against today’s daily build (r20089).

   9400configure-args.0.patch (8.1 KiB)

Would it be possible to remove –atype, and fuse it with –type? You can’t have both at the same time, so –type could be replaced if “a” is given or something.

I’ sync-ed it with current build to fix some line number offsets and fuzz factor. I just have to upload the new file.
I can think to reorganize the added arguments. I’m not sure, but maybe I added the option atype since due to an overload between normal and advanced options, so it’s possible to use the same letters as indicated by the output in the intercative execution. But I need to check them.
Stay tuned!

I’m thinking if –type starts with “a” then it can’t be a normal build anymore, so overloading sounds possible to me.

Here it is the updated patch, sync’d against build r21259.
It supports following command line arguments (some of them belong to  FS#8842 ):

1. -target=TARGET
2. -ram=RAM
3. -type=TYPE
4. -language=LANG
5. -tts=ENGINE
6. -voice=VOICE
7. -ttsopts=OPTS
8. -encopts=OPTS
9. -rbdir=DIR
10. -ccache
11. -no-ccache
12. -help

Now –type accepts a string to select multiple options.
So, if I want to build voice, I will do:

  ../tools/configure --type=av ...

For the bootloader, I will do:

  ../tools/configure --type=b ...

For the backlight option (when allowed), I will do:

  ../tools/configure --type=ab ...

(According with my naming rule for the patch files, last number before the extension indicates the value to be used in patch -p. HTH)

   build-9400configure-args_v3.0... (12.2 KiB)

Great, gonna check this out

Sync’ed against current build (r21388).

   build-9400configure-args_v4.0... (12.6 KiB)

Seems to work well! I’m all for committing (I think I’ll do it soonish).

Alright, I have had a look.

Nothing against the patch itself, but we’re getting short on args. The current implementation is badly limited to 10 arguments. This will make adapting make reconf quite hard (it passes all arguments at once). Do you have some plans regarding that?

Also, I was having problems with voice builds. Well, I don’t have any voice engine installed, but maybe you can take another look at it?

Actually, I already saw that fancy use of the command line arguments. Maybe a simple “$@” can fix the problem, but I need to investigate.
Probably a clearer way to manage the arguments should be implemented. I will try to rewrite it without change the script logic.
Thanks for the warning.

That rewrite should probably done before this patch, what do you say?

It would be awesome if you volunteer for that. It’s obvious I fail at this thing :)

Absolutely yes.
The major problem is that I’m thinking to completely rewrite the script, but I don’t know the use of the different compiler options, so a slight mistake on reproducing the same use of them could lead to a big headaches. Anyway I will try.

Completely rewriting seems a bit exaggerated. Is there any need to?

Ok, “completely” is really exaggerated. I will concentrate just on the arguments.

Sync’ed against current build (r21475).
No changes introduced yet about command line arguments as discussed earlier.

   build-9400configure-args_v5.0... (13 KiB)

This patch, applied after “build-9400configure-args_v5.0.patch” of the previous comment, normalize the script indentation ( vim: :set shiftwidth=4 tabstop=4 expandtab).

   build-9400configure-args_v5-i... (142.2 KiB)

Any progress?

PS: We don’t want the indentation changes I think.

I’m sorry, Thomas, I started working hard again, so my spare time is reduced!
I hope to complete the arguments management soon.
OK, for the indentation, I did it just because I found it a little bit fuzzy!

Here it is the “final” patch.
I’m not sure about the use of the options –ccache and –no-ccache, but I think to be maintained the original behaviour.
Note: some options work together (i.e. –type=av –encopts=’-c 10 -q 10’). If you want to leave an option value empty, set it to a blank, like –type=av –ttsopts=’ '

   build-9400configure-args_v6.0... (15.3 KiB)

I’ll have a look!