Ok, so I have a small minor PR I'd like to commit, and I'm not familiar with the cleanest way to do

Ok, so I have a small minor PR I'd like to commit, and I'm not familiar with the cleanest way to do that. At work, its make a branch, make the change, make a PR, and wait for acceptance. But I can't even push to a branch off this repo, so how does one commit back?

WaffleophagusOP•7/30/23, 11:26 PM

I could open an issue? But man, its so minor, I'd like to just tackle it.

Ttn nowhere in my container build scripts is waybar-git mentioned

Kyle Gospo•7/30/23, 11:41 PM

My guess is something that you are installing has a hard requirement for waybar

Kyle Gospo•7/30/23, 11:41 PM

And rather than remove it, it's installing an alternative that satisfies the dependency

Kyle Gospo•7/30/23, 11:41 PM

What you may want to do is try doing the same command on a live system

Kyle Gospo•7/30/23, 11:41 PM

See if it acts any differently and if it removes more packages than just the one you requested

tn•7/30/23, 11:42 PM

I think sway is requiring it since my image is based on sericea

tn•7/30/23, 11:42 PM

Trying to build with the swap happening in post after sway is removed

tn•7/30/23, 11:42 PM

Will see if that works

tn•7/30/23, 11:42 PM

Yep

tn•7/30/23, 11:43 PM

I feel like sway shouldn't have a hard requirement on waybar but sure

Ttn I think sway is requiring it since my image is based on sericea

Gerblesh•7/30/23, 11:43 PM

waybar is already installed in sericea

tn•7/30/23, 11:43 PM

Yes

Gerblesh•7/30/23, 11:43 PM

sway itself uses swaybar

tn•7/30/23, 11:43 PM

I'm override removing waybar and rpm-ostree is replacing it without me asking for it to

Gerblesh•7/30/23, 11:43 PM

So there's no dependency on waybar itself, just that the sway spin uses it

tn•7/30/23, 11:44 PM

Then idk why I can't remove it before removing sway

Ttn I'm override removing waybar and rpm-ostree is replacing it without me asking fo...

Gerblesh•7/30/23, 11:44 PM

You can just install sway on top of main/base

Gerblesh•7/30/23, 11:44 PM

That's what I did

tn•7/30/23, 11:45 PM

I don't want sway but I don't want to replicate the utils sericea has myself

tn•7/30/23, 11:45 PM

What's weird is that waybar-hyprland should be marked as providing waybar

tn•7/30/23, 11:46 PM

So idk why the original replace command I was doing was failing

tn•7/30/23, 11:46 PM

The solver should just know it can do that replacement

tn•7/30/23, 11:46 PM

Well it's fixed now anyway

tn•7/30/23, 11:47 PM

Well the image builds at least

Ttn I don't want sway but I don't want to replicate the utils sericea has myself

Gerblesh•7/30/23, 11:48 PM

You should install Hyprland on top of base

tn•7/30/23, 11:49 PM

Installing onto my machine results in

error: importing RPMs: creating importer: Processing file-overrides for package waybar-hyprland: Invalid absolute filepath 'waybar-hyprland-0.9.20.tar.gz'

error: importing RPMs: creating importer: Processing file-overrides for package waybar-hyprland: Invalid absolute filepath 'waybar-hyprland-0.9.20.tar.gz'

Gerblesh•7/30/23, 11:49 PM

Using sericea as a base isn't a great plan

Ttn Installing onto my machine results in `error: importing RPMs: creating importer:...

tn•7/30/23, 11:49 PM

Probably from the override command I ran locally earlier

Gerblesh•7/30/23, 11:49 PM

You want an override replace.

tn•7/30/23, 11:50 PM

Yes that's what I did

tn•7/30/23, 11:50 PM

Override replace succeeded locally but didn't actually replace the package, likely for same reasons as the build failing before

tn•7/30/23, 11:55 PM

sway-config-fedora requires waybar

tn•7/30/23, 11:55 PM

that was the issue

tn•7/30/23, 11:56 PM

I'll look at building from a clean base but I'll probably end up copying the majority of sericea's package list

Gerblesh•7/30/23, 11:57 PM

I use sway and I use different utils from sericea

Gerblesh•7/30/23, 11:57 PM

In fact you can do everything with one program if you wanted to

Ttn sway-config-fedora requires waybar

Kyle Gospo•7/31/23, 12:07 AM

All of the sources for these packages are available as well

Kyle Gospo•7/31/23, 12:07 AM

So if starting from scratch is too hard you could simply build a modified version of that package locally or on copr

Kyle Gospo•7/31/23, 12:07 AM

And do a replace for that as well

tn•7/31/23, 12:09 AM

Well I have it fixed, i just had to do the swap later

tn•7/31/23, 12:09 AM

Now to figure out why discord ui has regressed to an unusable state on wayland

tn•7/31/23, 12:10 AM

I had it working before but now i can barely see what i'm typing

j0rge•7/31/23, 12:19 AM

https://www.chainguard.dev/unchained/wolfi-act-dynamic-github-actions-from-wolfi-packages

wolfi-act: Dynamic GitHub Actions from Wolfi packages

Everything you need to know about securing the software supply chain.

j0rge•7/31/23, 12:19 AM

DOPE!

tulip•7/31/23, 12:26 AM

Hell yeah! That should make some actions run a bunch faster (plus the security benefits of using wolfi!!!)

j0rge•7/31/23, 12:43 AM

@bsherman you happy with this? https://github.com/ublue-os/config/pull/81

GitHub

feat: add repair and uninstall unused flatpak maintenance tasks by ...

Improves upon the existing flatpak update service unit.
closes #14

Jj0rge @bsherman you happy with this? https://github.com/ublue-os/config/pull/81

bsherman•7/31/23, 12:44 AM

The implementation is fine. But I asked for community vote above, though I think it got lost in chatter.

Jj0rge @bsherman you happy with this? https://github.com/ublue-os/config/pull/81

bsherman•7/31/23, 12:45 AM

I'm indifferent about it. I was just trying to be helping with that issue for investigating

j0rge•7/31/23, 12:57 AM

alright let's leave it open for a while then, no rush

WWaffleophagus Ok, so I have a small minor PR I'd like to commit, and I'm not familiar with the...

WaffleophagusOP•7/31/23, 1:05 AM

@MOD I didn't know who else to ping on this one. But could I get an answer on how to contribute back on some more key repos? I read through the documentation and they don't appear to talk about PRs and the like.

Ok, so I have a small minor PR I'd like to commit, and I'm not familiar with the cleanest way to do

Similar Threads

Similar Threads

Similar Threads