Skip to content

perf config: fix bug in iskeychar()#79

Open
paulkim90 wants to merge 1 commit intokosslab-kr:masterfrom
paulkim90:develop
Open

perf config: fix bug in iskeychar()#79
paulkim90 wants to merge 1 commit intokosslab-kr:masterfrom
paulkim90:develop

Conversation

@paulkim90
Copy link
Collaborator

perf config: fix bug in parsing 'man..*' config

To add new man viewer, configs like 'man.<tool>.cmd',
'man.<tool>.path' can be set into config file (~/.perfconfig).
But parsing config file is stopped because the config variable
contains '.' character i.e.

If setting 'man.xman.cmd' into config file,

    [man]
        gman.cmd = gman

when launching perf an error message is printed like below.

    Fatal: bad config file line 11 in /home/taeung/.perfconfig

So modify iskeychar() function to decide '.' character
as key character parsing config file.

Acked-by: Namhyung Kim <namhyung@kernel.org>
Cc: Jiri Olsa <jolsa@kernel.org>
Signed-off-by: Paul Kim <wogns3250@gmail.com>

Signed-off-by: Paul Kim <wogns3250@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments