~coder_kalyan

California, USA

https://coderkalyan.com

High schooler · president at AmadorUAVs · software developer · open source software enthusiast · tech enthusiast. https://coderkalyan.com https://amadoruavs.com

~coder_kalyan/public-inbox

Last active 2 days ago
View more

Recent activity

Re: [PATCH v6] config: load config file from default location 7 hours ago

From Kalyan Sriram to ~emersion/soju-dev

On Mon Nov 30, 2020 at 7:01 AM PST, Simon Ser wrote:
> We shouldn't need to Stat() the default location or anything. We can
> just set the default as the initial value for the flag:
>
> configPath := "/etc/soju/config"
> flag.StringVar(&configPath, "config", configPath, "path to configuration
> file")
Correct me if I'm wrong, but wouldn't that fail if there is no file in
the default location and no config option is passed in either?
>
> > /etc/config/soju.scfg
>
> Would prefer /etc/soju/config.
Sure, I just realized i should have moved to /etc/soju instead of

[PATCH v2] widgets: compose: render hints from binding config 2 days ago

From Kalyan Sriram to ~sircmpwn/aerc

In the compose message review widget, attempt to render the hints
for possible commands (attach, send, postpone, etc) based on current
aerc bindings configuration rather than a hardcoded string.
---
Fixed a typo/function name change.
 config/bindings.go | 14 ++++++++++++++
 widgets/compose.go | 19 ++++++++++++++++---
 2 files changed, 30 insertions(+), 3 deletions(-)

diff --git a/config/bindings.go b/config/bindings.go
index a19d4f3..cc5e5a8 100644
--- a/config/bindings.go
+++ b/config/bindings.go
@@ -59,6 +59,10 @@ func (bindings *KeyBindings) Add(binding *Binding) {
[message trimmed]

[PATCH] widgets: compose: render hints from binding config 3 days ago

From Kalyan Sriram to ~sircmpwn/aerc

In the compose message review widget, attempt to render the hints
for possible commands (attach, send, postpone, etc) based on current
aerc bindings configuration rather than a hardcoded string.
---
This is an attempt to render the suggestions in the message review
widget based on binds.conf. It's not perfect, and there's probably room
for improvement, formatting, and maybe configuration options? Posting
this to get some feedback.
 config/bindings.go | 14 ++++++++++++++
 widgets/compose.go | 19 ++++++++++++++++---
 2 files changed, 30 insertions(+), 3 deletions(-)

diff --git a/config/bindings.go b/config/bindings.go
index a19d4f3..cc5e5a8 100644
[message trimmed]

[PATCH] notmuch: trim excluded tags 3 days ago

From Kalyan Sriram to ~sircmpwn/aerc

Trims whitespace in list of excluded notmuch tags. This allows a comma
separated list with spaces to be correctly processed.
---
 worker/notmuch/worker.go | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/worker/notmuch/worker.go b/worker/notmuch/worker.go
index 67888ff..6281744 100644
--- a/worker/notmuch/worker.go
+++ b/worker/notmuch/worker.go
@@ -507,6 +507,9 @@ func (w *worker) loadExcludeTags(
		return nil
	}
	excludedTags := strings.Split(raw, ",")
[message trimmed]

[PATCH v6] config: load config file from default location 3 days ago

From Kalyan Sriram to ~emersion/soju-dev

If soju is run without a config option, it will check for a config
at /etc/config/soju.scfg. If this file is not found, and a config
option is not specified, default configuration options are loaded.
---
Apologies for so many patch versions, for some reason go.mod change
didn't get tracked.
 README.md           |  4 ++++
 cmd/soju/main.go    |  6 ++++++
 cmd/sojuctl/main.go |  6 ++++++
 config/config.go    | 10 ++++++++++
 doc/soju.1.scd      |  2 ++
 5 files changed, 28 insertions(+)

diff --git a/README.md b/README.md
[message trimmed]

[PATCH v5] config: load config file from default location 3 days ago

From Kalyan Sriram to ~emersion/soju-dev

If soju is run without a config option, it will check for a config
at /etc/config/soju.scfg. If this file is not found, and a config
option is not specified, default configuration options are loaded.
---
v5 cleans up dependencies now that we no longer use $XDG_CONFIG_HOME.
 README.md           |  4 ++++
 cmd/soju/main.go    |  6 ++++++
 cmd/sojuctl/main.go |  6 ++++++
 config/config.go    | 10 ++++++++++
 doc/soju.1.scd      |  2 ++
 go.mod              |  1 +
 6 files changed, 29 insertions(+)

diff --git a/README.md b/README.md
[message trimmed]

[PATCH v4] config: load config file from default location 3 days ago

From Kalyan Sriram to ~emersion/soju-dev

If soju is run without a config option, it will check for a config
at /etc/config/soju.scfg. If this file is not found, and a config
option is not specified, default configuration options are loaded.
---
v4 swaps out looking in XDG_CONFIG_HOME for /etc/soju
 README.md           |  4 ++++
 cmd/soju/main.go    |  6 ++++++
 cmd/sojuctl/main.go |  6 ++++++
 config/config.go    | 11 +++++++++++
 doc/soju.1.scd      |  2 ++
 go.mod              |  1 +
 go.sum              |  2 ++
 7 files changed, 32 insertions(+)
[message trimmed]

Re: [PATCH v3] config: load config file from default location 5 days ago

From Kalyan Sriram to ~emersion/soju-dev

On Thu Nov 26, 2020 at 10:23 AM PST, Simon Ser wrote:
>
> I wonder if it makes sense for soju to use user app directories.
>

> soju is a multi-user bouncer which listens (by default) on a fixed
> port. If there are multiple users on a server, instead of having one
> soju instance per user, you'll likely want a single soju instance for
> everybody.
>

Although I don't have a use case for a local soju instance per user,
we should keep in mind that it is *possible* that someone would want to
set this up without root privilege or for some other reason keep it

[PATCH] doc: aerc-config: fix typo 10 days ago

From Kalyan Sriram to ~sircmpwn/aerc

From: Kalyan Sriram <coder.kalyan@gmail.com>

---
 doc/aerc-config.5.scd | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/aerc-config.5.scd b/doc/aerc-config.5.scd
index c96ea12..ee75439 100644
--- a/doc/aerc-config.5.scd
+++ b/doc/aerc-config.5.scd
@@ -620,7 +620,7 @@ following special keys are supported:
# SEE ALSO

*aerc*(1) *aerc-imap*(5) *aerc-smtp*(5) *aerc-maildir*(5) *aerc-sendmail*(5)
[message trimmed]