Re: psmisc: update checksum

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 149 bytes --]

New comment by xtraeme on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583543586

Comment:
hmm why?

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 520 bytes --]

New comment by ailiop-git on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583550336

Comment:
> hmm why?

No idea. The whole thing seems messy, code is maintained in gitlab only (the sf code is stale), while releases are done in sf only. Quite honestly I don't see the purpose behind psmisc at this point, it's mostly a legacy thing.

Do we really need to have it as a requirement for base-system? nothing else (from base) should be using any of those utils anway.

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 180 bytes --]

New comment by xtraeme on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583563811

Comment:
this seems a bit weird to me, anyway hf

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 174 bytes --]

New comment by xtraeme on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583588899

Comment:
we still need pkill -s0,1

iirc

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 196 bytes --]

New comment by xtraeme on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583591294

Comment:
@leahneukirchen how about this? can we get rid of this?

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 328 bytes --]

New comment by ailiop-git on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583602333

Comment:
> we still need pkill -s0,1

pkill is part of procps-ng. I think there may be a few packages/scripts outside base still using killall and we would need to add psmisc as a dependency.

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 166 bytes --]

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583606826

Comment:
i use killall once in a while

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 244 bytes --]

New comment by xtraeme on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583606863

Comment:
@ailiop-git I'm aware of this. We should be ready to get rid of this dependency.. ping @leahneukirchen 

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 211 bytes --]

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583606826

Comment:
i use killall once in a while (but i guess pkill can do the same thing...)

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 258 bytes --]

New comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583741314

Comment:
I think `fuser`, `killall` and `pstree` are tools users expect.

Doesn't have to be in base-system, however.

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 196 bytes --]

New comment by leahneukirchen on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583741378

Comment:
We could pin a gitlab tarball if that is easier.

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 587 bytes --]

New comment by ailiop-git on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583848390

Comment:
> I think `fuser`, `killall` and `pstree` are tools users expect.
> 
> Doesn't have to be in base-system, however.

Thanks for confirming. So I've sent PR #18971 to first remove psmisc as a dep from dracut (I couldn't find any usage of any of the psmisc utils in any tagged dracut release), and then subsequent PRs (#18973, #18974, #18975) that remove psmisc from base-{system,minimal,voidstrap}.

BTW, is base-minimal used for anything?

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 517 bytes --]

New comment by ailiop-git on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583848839

Comment:
> We could pin a gitlab tarball if that is easier.

Not sure what you mean here by pinning a tarball (cache it locally in void infra distfiles?)

It's not a big problem anyway, just that changing checksums w/o having a new release seems a bit sketchy/unhygienic release process from the side of psmisc, but oh well, the less we have rely on that pkg the better I guess.

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 238 bytes --]

New comment by xtraeme on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583848998

Comment:
@ailiop-git this is not simple as it sounds. If a checksum changed could mean malware, be careful

Re: psmisc: update checksum

[voidlinux-github]

[-- Attachment #1: Type: text/plain, Size: 555 bytes --]

New comment by ailiop-git on void-packages repository

https://github.com/void-linux/void-packages/pull/18823#issuecomment-583852130

Comment:
> @ailiop-git this is not simple as it sounds. If a checksum changed could mean malware, be careful

Totally agree (plus sourceforge has a history with malware..), which is why I'm trying to get rid of the pkg altogether. Unfortunately I haven't kept the previous two tarballs from the same release that changed cksums, in order to run a diff. I'll try to ping the upstream author anyway to see if he's aware.