Skip to content

Commit

Permalink
Revert "config: free include path on successful load"
Browse files Browse the repository at this point in the history
This reverts commit 9245088.
  • Loading branch information
ddevault committed Jul 17, 2018
1 parent 9c8d73d commit db3a363
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion sway/config.c
Original file line number Diff line number Diff line change
Expand Up @@ -474,7 +474,6 @@ static bool load_include_config(const char *path, const char *parent_dir,
list_del(config->config_chain, index);
return false;
}
free(real_path);

// restore current_config_path
config->current_config_path = parent_config;
Expand Down

2 comments on commit db3a363

@ddevault
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have time to debug it in depth right now but this commit broke my includes, @ianyfan. My config is here:

https://git.sr.ht/~sircmpwn/dotfiles/tree/.config/sway/config#n181

My output config in ~/.config/sway/config.d/:hostname/outputs doesn't get loaded.

@ianyfan
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird. I'm happy to leave it for now though.

Please sign in to comment.