[manjaro-tools.conf] iso_name useless var? #163

Closed
opened 2015-12-18 22:49:13 +00:00 by philm · 1 comment
philm commented 2015-12-18 22:49:13 +00:00 (Migrated from gitlab2.manjaro.org)

Created by: udeved

As it is now, we have two very similar variables, that is
iso_name
dist_name

We currently mix them in code.
Is there any reason we shouldn't remove iso_name from configuration?

Usecase is:

iso_name=manjaro
dist_name=Manjaro

iso_name=sonar
dist_name=Sonar

We can achieve the same by converting dist_name to lower cases, and let iso_name be non configurable, but determined by dist_name.

*Created by: udeved* As it is now, we have two very similar variables, that is iso_name dist_name We currently mix them in code. Is there any reason we shouldn't remove iso_name from configuration? Usecase is: iso_name=manjaro dist_name=Manjaro iso_name=sonar dist_name=Sonar We can achieve the same by converting dist_name to lower cases, and let iso_name be non configurable, but determined by dist_name.
philm commented 2015-12-19 09:03:20 +00:00 (Migrated from gitlab2.manjaro.org)

Created by: philmmanjaro

Fine. Simplify it then ...

*Created by: philmmanjaro* Fine. Simplify it then ...
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
tools/manjaro-tools#163
No description provided.