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

refactor desktops #8

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

refactor desktops #8

wants to merge 9 commits into from

Conversation

Misterio77
Copy link
Member

  • refactor(hosts/common/desktop): tirar comentários desnecessários
  • refactor(hosts/common/desktop): importar common/default, cleanup
  • fix(hosts): especificar hostPlatform nos hardware-configuration.nix
  • chore(hosts/common/desktop): cleanup
  • chore(treewide): formatar tudo
  • fix(hosts): deixar stateVersion per hosts
  • chore(hosts/desktop): retirar config redundante
  • refactor(treewide): mover usuários, reorganizar commons
  • fix(flake): retirar leftover nix-flatpak

@Misterio77 Misterio77 requested a review from LuNeder August 29, 2024 13:35
@Misterio77
Copy link
Member Author

Dá um bizu e vê se faz sentido. Em tese não é pra ter mudado a funcionalidade de nada.

@Misterio77
Copy link
Member Author

(Recomendo olhar cada commit individualmente)

E, also, quem é tiago? KKKKK

@Misterio77 Misterio77 force-pushed the refactor-desktops branch 2 times, most recently from 6a8ac5b to 54fd89f Compare August 29, 2024 13:50
./gelosentity.nix
./luana.nix
./misterio.nix
./tiago.nix
Copy link
Contributor

Choose a reason for hiding this comment

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

Aqui ta errado, é pra importar o Tiago só no Rockhopper e o GELOS Entity só no Macaroni
Ter os dois não faz sentido, já que ambos são a “conta GELOS” em que ta clonada a config

{
users.users.gelosentity = {
isNormalUser = true;
description = "GELOS-Entity"; # infra (nixos config) cloned on this user's home
Copy link
Contributor

Choose a reason for hiding this comment

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

Não sei se eu gosto disso separado da file de cada aio, já que eles estão clonados em users diferentes (no rockhopper no tiago e no macaroni na GELOS-Entity)

Copy link
Contributor

Choose a reason for hiding this comment

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

Talvez pudessemos manter a “conta GELOS”/“admin” dentro do arquivo, e aí os outros users nessa common/users como vc fez msm?

Talvez possibilitando também importar os users individualmente ao invés de em ambos os aios pela users/default.nix (ou talvez deixando isso assim mesmo, sei lá se ia fazer sentido pra alguém além do Tiago e da GELOS-Entity querer estar em um dos aios só ksjsksjs)

@@ -2,44 +2,17 @@

{
imports = [
./default.nix
Copy link
Contributor

@LuNeder LuNeder Aug 29, 2024

Choose a reason for hiding this comment

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

Aqui eu tinha intencionalmente feito umas configurações diferentes da desse aí, como por exemplo pedir senha pro sudo e sempre mostrar a mensagenzinha com o pedido da senha

Não sei se é uma boa ideia deixar assim sem pedir senha, já que é um pc semi-publico e tal seria uma boa ideia ter essa confirmação acho.

Acho tbm que mutableUsers = false; não faz sentido pra esses PCs, vai complicar mto a vida de quem quer mudar a senha e tal

environment.systemPackages = [
pkgs.curl
pkgs.git
Copy link
Contributor

Choose a reason for hiding this comment

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

Pq tirou o git?

@@ -7,17 +7,12 @@
];

networking.networkmanager.enable = true;
networking.networkmanager.ethernet.macAddress = "permanent"; # use real Mac address
Copy link
Contributor

Choose a reason for hiding this comment

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

Aqui acho que a gente vai precisar disso se for pedir ethernet pra STI né?

{
users.users.tiago = {
isNormalUser = true;
description = "GELOS-Tiago"; # infra (nixos config) cloned on this user's home
Copy link
Contributor

Choose a reason for hiding this comment

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

Mesma coisa que falei pra GELOS-Entity

@@ -30,7 +30,6 @@
outputs = {
self,
nixpkgs,
nix-flatpak,
Copy link
Contributor

Choose a reason for hiding this comment

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

Ctz que o inputs.nix-flatpak do outro arquivo lá ainda funciona?

# Sudo
security.sudo = {
extraConfig = ''
Defaults lecture = always
Copy link
Contributor

Choose a reason for hiding this comment

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

Isso aqui não possui efeito se o sudo não estiver pedindo senha

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, e essa porra n vai evaluar não pq tem 2 extraConfig

Pode meter um mkForce ali que resolve os dois problemas (acho): passa a evaluar e volta a pedir senha

@@ -1,32 +1,39 @@
# Do not modify this file! It was generated by ‘nixos-generate-config’
Copy link
Contributor

Choose a reason for hiding this comment

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

Lmao, mas eu confio que vc sabe oq ta fazendo :)

...
}: {
imports = [
../global
Copy link
Contributor

@LuNeder LuNeder Aug 29, 2024

Choose a reason for hiding this comment

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

Não sei se é uma boa ideia importar o global aqui, pelos motivos que falei por aqui e tal

O sudo sem senha é um exemplo disso acho, como é um pc meio semi-publico acho que seria legal ter um “olha a merda que você vai fazer!” antes de deixar mudar a senha do coleguinha. Um fator mais psicológico, talvez, mas ainda assim uma barreira.

Outro exemplo disso é o immutableUsers que o common/global tem e não faz mto sentido ter aqui

@LuNeder
Copy link
Contributor

LuNeder commented Aug 29, 2024

E, also, quem é tiago? KKKKK

Amigo do Furry

@LuNeder
Copy link
Contributor

LuNeder commented Aug 29, 2024

Tem uns ali marcado como “outdated” por causa do rename, mas lê eles tbm pq não tão outdated não skjssksks

@LuNeder
Copy link
Contributor

LuNeder commented Aug 29, 2024

Em tese não é pra ter mudado a funcionalidade de nada.

Mudou algumas coisas sim, botou um user a mais nos dois aio, o sudo não pede mais senha, os usuários viraram imutáveis e talvez mais alguma coisa que eu não tenha percebido? N sei

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.

2 participants