Looking good. I wouldn't mind a leftover file in `/etc/profile.d/?` or `~/.config/autostart`, but ma

Xxyny Looking good. I wouldn't mind a leftover file in `/etc/profile.d/?` or `~/.confi...

A

Arcitec•5/12/23, 5:37 PM

Great to hear.

Yeah the "it won't disable everything" is what I dislike too.

I am also unsure what actually happens when there is a dead symlink in

~/.config/autostart

~/.config/autostart

. If the OS complains or just moves along like nothing.

Either way I am trying to think of ways to make sure the users see zero errors if they move between flavors of ublue. Some that use it and some that don't.

A

Arcitec•5/12/23, 5:39 PM

I'll do some tests in a VM to see what's the impact of dead autostart symlinks.

X

xynyOP•5/12/23, 5:40 PM

I feel like startingpoint is evolving so much (into a way more customizable durection, though), that it's kinda losing my original vision for it; being a lean and thin template where all code is thoroughly documented, easy to grow and stretch, but understandable in it's entirety. It's turned more into a framework-as-a-template, or it's turning. from kickstart.nvim to NvChad.

X

xynyOP•5/12/23, 5:40 PM

But maybe I don't mind that. I'd just like to make sure that every bit of it's code is documented well.

A

Arcitec•5/12/23, 5:40 PM

It's still lean and thin. Thinner than before

A

Arcitec•5/12/23, 5:41 PM

No more need to edit build.sh. And soon being able to remove yafti if wanted. Leaner in other words

X

xynyOP•5/12/23, 5:41 PM

Well, it was arguably at it's thinnest when the recipe had two config values;

rpms:

rpms:

rpms:

rpms: and

flatpaks:

flatpaks:

flatpaks:

flatpaks:.

A

Arcitec•5/12/23, 5:41 PM

Yeah but then it was too opinionated and pre-installed stuff, so swings and roundabouts.

B

bsherman•5/12/23, 5:42 PM

I think startingpoint is still pretty lean

especially compared to bluefin or my custom image, etc

X

xynyOP•5/12/23, 5:42 PM

And sure, you can do more things from the

recipe.yml

recipe.yml

recipe.yml

recipe.yml and you don't have to get into the weeds, the codebase is just a bit7 more complicated

Bbsherman I think startingpoint is still pretty lean 😉 especially compared to bluefin o...

X

xynyOP•5/12/23, 5:42 PM

Sure, it's a startingpoint after all

A

Arcitec•5/12/23, 5:43 PM

Yeah, the autorun functionality might be what you're thinking of. It's pretty easy to understand it though.

A

Arcitec•5/12/23, 5:43 PM

scripts:
  # "Pre" scripts run very early in the build, immediately after your custom
  # repos have been imported (so that you can access those repos if necessary).
  pre:
    # Automatically runs script files within "scripts/pre/".
    - autorun.sh
    # Manually listed scripts.
    # - example_pre.sh

  # "Post" scripts run at the end of the build process.
  post:
    # Automatically runs script files within "scripts/post/".
    - autorun.sh
    # Manually listed scripts.
    # - example_post.sh

scripts:
  # "Pre" scripts run very early in the build, immediately after your custom
  # repos have been imported (so that you can access those repos if necessary).
  pre:
    # Automatically runs script files within "scripts/pre/".
    - autorun.sh
    # Manually listed scripts.
    # - example_pre.sh

  # "Post" scripts run at the end of the build process.
  post:
    # Automatically runs script files within "scripts/post/".
    - autorun.sh
    # Manually listed scripts.
    # - example_post.sh

scripts:
  # "Pre" scripts run very early in the build, immediately after your custom
  # repos have been imported (so that you can access those repos if necessary).
  pre:
    # Automatically runs script files within "scripts/pre/".
    - autorun.sh
    # Manually listed scripts.
    # - example_pre.sh

  # "Post" scripts run at the end of the build process.
  post:
    # Automatically runs script files within "scripts/post/".
    - autorun.sh
    # Manually listed scripts.
    # - example_post.sh

scripts:
  # "Pre" scripts run very early in the build, immediately after your custom
  # repos have been imported (so that you can access those repos if necessary).
  pre:
    # Automatically runs script files within "scripts/pre/".
    - autorun.sh
    # Manually listed scripts.
    # - example_pre.sh

  # "Post" scripts run at the end of the build process.
  post:
    # Automatically runs script files within "scripts/post/".
    - autorun.sh
    # Manually listed scripts.
    # - example_post.sh

A

Arcitec•5/12/23, 5:43 PM

Plus the great

scripts/README.md

scripts/README.md

scripts/README.md

scripts/README.md file which explains in depth.

https://github.com/ublue-os/startingpoint/blob/main/scripts/README.md

A

Arcitec•5/12/23, 5:43 PM

I don't see any issues with it.

What could be improved?

X

xynyOP•5/12/23, 5:45 PM

It's pretty good. Good job Arcitect, always looking to improve everything. We'll get to the moon and back with that attitude in no time.

A

Arcitec•5/12/23, 5:46 PM

Thank you too.

Oh and regarding the autostart script, I'll get to testing the dead symlinks now to see what the OS says!

Xxyny It's pretty good. Good job Arcitect, always looking to improve everything. We'll...

B

bsherman•5/12/23, 5:46 PM

the moon? lets go to Mars!

Bbsherman the moon? lets go to Mars!

X

xynyOP•5/12/23, 5:48 PM

I was bout to write that, but then I remembered this asset gamblers and startup founders saying of To the moon , routinely sarcastically used by Ben Awad on YT.

A

Arcitec•5/12/23, 5:50 PM

@hermmm Alright so at least on

gnome-session

gnome-session

gnome-session

gnome-session it only leads to a silent syslog entry:

WARNING: Desktop file /var/home/johnny/.config/autostart/foo.desktop for application foo.desktop could not be parsed or references a missing TryExec binary

WARNING: Desktop file /var/home/johnny/.config/autostart/foo.desktop for application foo.desktop could not be parsed or references a missing TryExec binary

WARNING: Desktop file /var/home/johnny/.config/autostart/foo.desktop for application foo.desktop could not be parsed or references a missing TryExec binary

WARNING: Desktop file /var/home/johnny/.config/autostart/foo.desktop for application foo.desktop could not be parsed or references a missing TryExec binary

I'll try KDE too. If both are fine with missing links, I'd say it's tested enough to be okay leaving an orphan?

Edit: KDE gives a silent syslog warning too, from

systemd-xdg-aut

systemd-xdg-aut

systemd-xdg-aut

systemd-xdg-aut which says

/var/home/johnny/.config/autostart/foo.desktop: stat() failed, ignoring: No such file or directory

/var/home/johnny/.config/autostart/foo.desktop: stat() failed, ignoring: No such file or directory

/var/home/johnny/.config/autostart/foo.desktop: stat() failed, ignoring: No such file or directory

/var/home/johnny/.config/autostart/foo.desktop: stat() failed, ignoring: No such file or directory

B

bsherman•5/12/23, 5:50 PM

@Arcitec I'll add this to the paths proposal discussion after we chat but... how passionate are you about the "no clunky paths"?

I'm looking at how to move from

/usr/share/ublue-os/ublue-os-update-services

/usr/share/ublue-os/ublue-os-update-services

/usr/share/ublue-os/ublue-os-update-services

/usr/share/ublue-os/ublue-os-update-services to

/usr/share/ublue-os/update-services

/usr/share/ublue-os/update-services

/usr/share/ublue-os/update-services

/usr/share/ublue-os/update-services and I don't like what I'm seeing.

It's due to the RPM spec, and I really like the cleanliness of the spec as is.

See this line for example: https://github.com/ublue-os/config/blob/main/rpmspec/ublue-os-update-services.spec#L49

%attr(0644,root,root) %{_datadir}/%{VENDOR}/%{NAME}/%{_sysconfdir}/rpm-ostreed.conf

%attr(0644,root,root) %{_datadir}/%{VENDOR}/%{NAME}/%{_sysconfdir}/rpm-ostreed.conf

%attr(0644,root,root) %{_datadir}/%{VENDOR}/%{NAME}/%{_sysconfdir}/rpm-ostreed.conf

%attr(0644,root,root) %{_datadir}/%{VENDOR}/%{NAME}/%{_sysconfdir}/rpm-ostreed.conf

The

%{VENDOR}

%{VENDOR}

%{VENDOR}

%{VENDOR} is

ublue-os

ublue-os

ublue-os

ublue-os and the

%{NAME}

%{NAME}

%{NAME}

%{NAME} is

ublue-os-update-services

ublue-os-update-services

ublue-os-update-services

ublue-os-update-services.

edit: add local context

AArcitec @hermmm Alright so at least on `gnome-session` it only leads to a silent syslog ...

X

xynyOP•5/12/23, 5:50 PM

Sure!

Bbsherman @Arcitec I'll add this to the paths proposal discussion after we chat but... how...

B

bsherman•5/12/23, 5:52 PM

I don't want the re-name the RPM as it should be

ublue-os-update-services

ublue-os-update-services

ublue-os-update-services

ublue-os-update-services in the RPM database.

Bbsherman @Arcitec I'll add this to the paths proposal discussion after we chat but... how...

A

Arcitec•5/12/23, 5:52 PM

Ew. I see the point but I think there's a way in RPM specs to create dynamic variables, where we could add one that's something like

%SHARED_DATA_PATH=${NAME:/-ublue-os/}

%SHARED_DATA_PATH=${NAME:/-ublue-os/}

%SHARED_DATA_PATH=${NAME:/-ublue-os/}

%SHARED_DATA_PATH=${NAME:/-ublue-os/} (pseudocode)

A

Arcitec•5/12/23, 5:52 PM

I remember seeing people do something dynamic like that to get the first 7 characters of Git commit hashes

AArcitec Ew. I see the point but I think there's a way in RPM specs to create dynamic var...

B

bsherman•5/12/23, 5:53 PM

rpmspec is not my strength, I'm definitely learning

A

Arcitec•5/12/23, 5:54 PM

There is some way, similar to bash variable substitution. So I think it's worth trying to solve that. Dynamically cutting away the

ublue-os-

ublue-os-

ublue-os-

ublue-os- prefix at that location. We should avoid the redundancy, and we're referring to the full path in other places such as

main

main

.

B

bsherman•5/12/23, 5:54 PM

yeah, I'll read up https://rpm-packaging-guide.github.io/

A

Arcitec•5/12/23, 5:55 PM

I will try to find a spec that does it.

A

Arcitec•5/12/23, 5:55 PM

lol nvm, COPR is down https://copr-dist-git.fedorainfracloud.org/cgit/sunwire/virt-manager/virt-manager.git

A

Arcitec•5/12/23, 5:55 PM

That spec does it but I can't access it :/

B

bsherman•5/12/23, 5:55 PM

I like the idea of maybe having a "UBLUE_SUB_NAME" which conditionally removes any "ublue-os-" prefix from "%{NAME}" otherwise uses %{NAME}

A

Arcitec•5/12/23, 5:56 PM

Yeah and if we can have a nice pattern to auto-generate it, we'll be set for all rpmspecs

B

bsherman•5/12/23, 5:58 PM

i do want to make the point... this is basically vanity, though

A

Arcitec•5/12/23, 5:59 PM

It is but it's such nice and fresh vanity / bling.

A

Arcitec•5/12/23, 6:00 PM

I am searching too, to help out.

B

bsherman•5/12/23, 6:00 PM

appreciate it

A

Arcitec•5/12/23, 6:00 PM

I found it

B

bsherman•5/12/23, 6:01 PM

if reasonably easy, and still clean/templated like our specs are, i'm happy to do it... but if it's super jank or becomes a lot of hard coding, i would push back... reasonable?

A

Arcitec•5/12/23, 6:01 PM

https://rpm-software-management.github.io/rpm/manual/macros.html

%define <name>[(opts)] <body>

%define <name>[(opts)] <body>

%define <name>[(opts)] <body>

%define <name>[(opts)] <body>

With this function:

%{gsub:...}    replace occurences of pattern in a string
                        (see Lua `string.gsub()`)

%{gsub:...}    replace occurences of pattern in a string
                        (see Lua `string.gsub()`)

%{gsub:...}    replace occurences of pattern in a string
                        (see Lua `string.gsub()`)

%{gsub:...}    replace occurences of pattern in a string
                        (see Lua `string.gsub()`)

rpm.org - Macro syntax

B

bsherman•5/12/23, 6:01 PM

ah, it's the %define

A

Arcitec•5/12/23, 6:01 PM

I don't know exactly how to use it though, but that's the one. It does a string substitute and only changes if it finds it, else it uses the original.

B

bsherman•5/12/23, 6:01 PM

ok, i'll play with this

A

Arcitec•5/12/23, 6:01 PM

Since it's a LUA pattern, this would do for the pattern:

^ublue-os-

^ublue-os-

^ublue-os-

^ublue-os-

B

bsherman•5/12/23, 6:02 PM

i think i just hit that in other docs source

A

Arcitec•5/12/23, 6:02 PM

yea

A

Arcitec•5/12/23, 6:03 PM

@bsherman

string.gsub("ublue-os-startup-scripts", "^ublue%-os%-", "")
startup-scripts    1

string.gsub("ublue-os-startup-scripts", "^ublue%-os%-", "")
startup-scripts    1

string.gsub("ublue-os-startup-scripts", "^ublue%-os%-", "")
startup-scripts    1

string.gsub("ublue-os-startup-scripts", "^ublue%-os%-", "")
startup-scripts    1

A

Arcitec•5/12/23, 6:04 PM

That's the exact pattern we'll need to use, since lua interprets plain dashes as character ranges (lol).

A

Arcitec•5/12/23, 6:04 PM

"^ublue%-os%-"

"^ublue%-os%-"

"^ublue%-os%-"

"^ublue%-os%-"

A

Arcitec•5/12/23, 6:07 PM

After reading a bit of the macro examples, I assume that it would look similar to this in use:

%define project_name %{gsub:"^ublue%-os%-", ""}

%define project_name %{gsub:"^ublue%-os%-", ""}

%define project_name %{gsub:"^ublue%-os%-", ""}

%define project_name %{gsub:"^ublue%-os%-", ""}