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

New log format. #867

Open
Sk7Str1p3 opened this issue Nov 9, 2024 · 6 comments
Open

New log format. #867

Sk7Str1p3 opened this issue Nov 9, 2024 · 6 comments
Labels
enhancement New feature or request

Comments

@Sk7Str1p3
Copy link
Contributor

Format of logs rn:

+ blkid /dev/mapper/crypted -o export
+ grep -q '^TYPE='
+ mkfs.btrfs /dev/mapper/crypted -f 
+ rm -rf /tmp/tmp.tsScPEKz1v
+ device=/dev/vdb
+ imageName=main
+ imageSize=2G 
+ name=main
+ type=disk
+ device=/dev/vdb
+ efiGptPartitionFirst=1
+ type=gpt

Is unreadable and ugly asf. It looks not like log but bash script. I wish it could be like nixos-rebuild log or rather home-manager or best of both implementations.

@Sk7Str1p3
Copy link
Contributor Author

Just give me file generating logs and I'll send you PR in a week

@iFreilicht
Copy link
Contributor

It looks not like log but bash script.

That's because it is. These logs are generated by set -x.

So you'd have to write separate logging statements for every shell invocation, which would make the logs look nicer but the shell script twice as long and way less readable.

Better logging is being implemented in #789

@iFreilicht iFreilicht added the enhancement New feature or request label Nov 10, 2024
@Sk7Str1p3
Copy link
Contributor Author

@iFreilicht I made few progress with my project, cli part is ready, few more days and current Disko version (with current features I mean) will be ready to be merged. If you wish me continue, create separate branch for this and I'll make a PR asap. Thx

@iFreilicht
Copy link
Contributor

Sure, I'd be glad to take a look at it!

@Sk7Str1p3
Copy link
Contributor Author

The only reason I will not fork project and help with writing this with python is because my project is obviously doomed to oblivion, no one will not even know about it. Even if I manage to make a nixpkg everyone will still use your version. I hope you understand that too and don't think I can talk only. Wait my first commit next week.

@Mic92
Copy link
Member

Mic92 commented Nov 23, 2024

Makes sense. You don't need to prove anything to us. We know it's technically possible to write it it in C++, we just prefer not to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants