Danny, > Hey Par, try this patch, it should also fix the issue with qos' of > different case. Also catches the issue where the env var is set. > This patch will be in 2.2.4.
This also works. I actually wrote exactly this patch as my first attempt, minus the QOS part. Another good thing with strcasecmp() is that it will still work if the opposite problem happen and a upper-case account get into the system somehow. Main reason I changed my mind and used xstrtolower() was to be more consistent with slurmctld. Multiple calls to strcmp() should also be slightly faster than strcasecmp() but probably nothing that will be noticeable. What env var are you talking about? /Pär
