Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Add systemd logger #1654

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from
Draft

Feature: Add systemd logger #1654

wants to merge 13 commits into from

Conversation

glensc
Copy link
Collaborator

@glensc glensc commented Dec 5, 2023

@glensc glensc self-assigned this Dec 5, 2023
@glensc
Copy link
Collaborator Author

glensc commented Dec 5, 2023

Not sure how to setup the extras, so systemd would not be attempted to install by default:

@glensc
Copy link
Collaborator Author

glensc commented Dec 5, 2023

and systemd not available on alpine:

/ # apk add systemd
ERROR: unable to select packages:
  systemd (no such package):
    required by: world[systemd]

so, need to setup the marker to install only on non-alpine linux, or libc linuxes or somehow make it optional.

@glensc
Copy link
Collaborator Author

glensc commented Dec 5, 2023

here's pep what could be filtered with markers:

not much useful

marker alpine debian
os_name posix posix
sys.platform linux linux
platform.release() 6.4.16-linuxkit 6.1.0-rpi6-rpi-v8
sys.implementation.name cpython cpython
platform.machine() aarch64 aarch64
platform.python_implementation() CPython CPython

also:

@glensc
Copy link
Collaborator Author

glensc commented Dec 5, 2023

Dump of modifying the journal in display mode

image

@glensc
Copy link
Collaborator Author

glensc commented Dec 11, 2023

@simonc56 what's your interest in this? i.e. why the eyes?

@glensc
Copy link
Collaborator Author

glensc commented Jan 14, 2024

A way to set is optional = True:

tree_sitter_languages = {version = ">=1.7.0", optional = true}

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