Implement template support

[LaakkonenJussi]

Add two patches which change profile loading to be delayed until all
arguments are parsed. This is due to the template key replacement, since
templates are to be given as arguments as well. Each line in the profile
line is checked and if one template is found they are replaced with the
corresponding value. If no changes are required the profile line is
untouched.

[LaakkonenJussi]

@okodron review is appreciated.

For @Tomin1 and @spiiroin please see also LaakkonenJussi/firejail#1 which contains the changes made here as directly on top of Sailfish OS patches + upstream git fork.

[Tomin1]

This works for what we currently need. I think we may need to revisit those allowed characters a little if we use these for directory paths later.

[LaakkonenJussi]

This works for what we currently need. I think we may need to revisit those allowed characters a little if we use these for directory paths later.

Could you be more specific if you have some concerns somewhere? That part could be improved anyways a bit.

[Tomin1]

This works for what we currently need. I think we may need to revisit those allowed characters a little if we use these for directory paths later.

Could you be more specific if you have some concerns somewhere? That part could be improved anyways a bit.

So, IIUC this doesn't allow dashes but I think we do want to keep them in directory names. Otherwise it's probably fine.

[LaakkonenJussi]

This works for what we currently need. I think we may need to revisit those allowed characters a little if we use these for directory paths later.

Could you be more specific if you have some concerns somewhere? That part could be improved anyways a bit.

So, IIUC this doesn't allow dashes but I think we do want to keep them in directory names. Otherwise it's probably fine.

I could add them as supported chars since the actual D-Bus names are checked by the dbus.c:dbus_check_name() - and I guess those are allowed in sailjail.

[Tomin1]

This works for what we currently need. I think we may need to revisit those allowed characters a little if we use these for directory paths later.

Could you be more specific if you have some concerns somewhere? That part could be improved anyways a bit.

So, IIUC this doesn't allow dashes but I think we do want to keep them in directory names. Otherwise it's probably fine.

I could add them as supported chars since the actual D-Bus names are checked by the dbus.c:dbus_check_name() - and I guess those are allowed in sailjail.

Looks like sailjail allows organization_name.application_name D-Bus service as is. I thought it would be doing some sort of substitution.

[LaakkonenJussi]

This works for what we currently need. I think we may need to revisit those allowed characters a little if we use these for directory paths later.

Could you be more specific if you have some concerns somewhere? That part could be improved anyways a bit.

So, IIUC this doesn't allow dashes but I think we do want to keep them in directory names. Otherwise it's probably fine.

I could add them as supported chars since the actual D-Bus names are checked by the dbus.c:dbus_check_name() - and I guess those are allowed in sailjail.

Looks like sailjail allows organization_name.application_name D-Bus service as is. I thought it would be doing some sort of substitution.

I did remove the substitutions as there was no proper reason to do that. And I added better way to this to handle chars _-/..

[LaakkonenJussi]

Since this does not depend on any other component in our case I'm merging this to be tested in devel.