Github messages for voidlinux
 help / color / mirror / Atom feed
* [ISSUE] spotify: SHA256 of EULA mismatching
@ 2021-01-17 23:24 waynevanson
  2021-01-18 18:34 ` teh-jazzman
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: waynevanson @ 2021-01-17 23:24 UTC (permalink / raw)
  To: ml

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

New issue by waynevanson on void-packages repository

https://github.com/void-linux/void-packages/issues/27998

Description:
<!-- Don't request update of package. We have a script for that. https://alpha.de.repo.voidlinux.org/void-updates/void-updates.txt . However, a quality pull request may help. -->

### System

* xuname:  
  Void 5.9.16_1 x86_64 GenuineIntel uptodate rrFFFFFFF
* package:  
 spotify-1.1.42_2

### Expected behavior

Installing package works without any errors.

### Actual behavior

`post_extract()` calls `message_error()` because the checksum of Spotify's EULA does not match the current.

```bash
post_extract() {
	curl -L https://www.spotify.com/us/legal/end-user-agreement/ |
		w3m -dump -I utf-8 -T text/html |
		sed -n '/Spotify Terms and Conditions of Use/,/^USA$/p' > EULA

	filesum="$(xbps-digest EULA)"
	if [ "$filesum" != "$_license_checksum" ]; then
		msg_error "SHA256 mismatch for EULA:\n$filesum\n"
	fi
}
```

### Steps to reproduce the behavior

- enable restricted packages
- `./xbps-src pkg spotify` in `void-packages/`

### Questions

@teh-jazzman What is the reason behind checking the EULA and throwing an error?

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: spotify: SHA256 of EULA mismatching
  2021-01-17 23:24 [ISSUE] spotify: SHA256 of EULA mismatching waynevanson
@ 2021-01-18 18:34 ` teh-jazzman
  2021-01-18 19:00 ` ericonr
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: teh-jazzman @ 2021-01-18 18:34 UTC (permalink / raw)
  To: ml

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

New comment by teh-jazzman on void-packages repository

https://github.com/void-linux/void-packages/issues/27998#issuecomment-762412577

Comment:
There was a discussion at #23576, which I missed, so I can't tell you. Maybe @ericonr can explain?

I assume it is to ensure that the downloaded license file is indeed a valid license and not, for example, a 404 error message or something like that. I assume since the package can be semi-automatically updated via void mechanisms it is necessary to provide a valid and up-to-date license file. Or it is to prevent the package from installing when there _might_ be something in the license that forbids that.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: spotify: SHA256 of EULA mismatching
  2021-01-17 23:24 [ISSUE] spotify: SHA256 of EULA mismatching waynevanson
  2021-01-18 18:34 ` teh-jazzman
@ 2021-01-18 19:00 ` ericonr
  2021-01-18 19:03 ` ericonr
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-01-18 19:00 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/issues/27998#issuecomment-762422451

Comment:
> Or it is to prevent the package from installing when there might be something in the license that forbids that.

Basically that. If the license changes, it should be acknowledged and the template should be fixed.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: spotify: SHA256 of EULA mismatching
  2021-01-17 23:24 [ISSUE] spotify: SHA256 of EULA mismatching waynevanson
  2021-01-18 18:34 ` teh-jazzman
  2021-01-18 19:00 ` ericonr
@ 2021-01-18 19:03 ` ericonr
  2021-01-18 21:09 ` Chocimier
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-01-18 19:03 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/issues/27998#issuecomment-762423745

Comment:
A PR to fix that should be pretty simple. To allow for easier visualization, include the license text in the PR so we can check it out.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: spotify: SHA256 of EULA mismatching
  2021-01-17 23:24 [ISSUE] spotify: SHA256 of EULA mismatching waynevanson
                   ` (2 preceding siblings ...)
  2021-01-18 19:03 ` ericonr
@ 2021-01-18 21:09 ` Chocimier
  2021-01-18 21:09 ` [ISSUE] [CLOSED] " Chocimier
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: Chocimier @ 2021-01-18 21:09 UTC (permalink / raw)
  To: ml

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

New comment by Chocimier on void-packages repository

https://github.com/void-linux/void-packages/issues/27998#issuecomment-762471489

Comment:
closed by #28022

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [ISSUE] [CLOSED] spotify: SHA256 of EULA mismatching
  2021-01-17 23:24 [ISSUE] spotify: SHA256 of EULA mismatching waynevanson
                   ` (3 preceding siblings ...)
  2021-01-18 21:09 ` Chocimier
@ 2021-01-18 21:09 ` Chocimier
  2021-10-08  5:23 ` Animeshz
  2021-10-09  7:06 ` teh-jazzman
  6 siblings, 0 replies; 8+ messages in thread
From: Chocimier @ 2021-01-18 21:09 UTC (permalink / raw)
  To: ml

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

Closed issue by waynevanson on void-packages repository

https://github.com/void-linux/void-packages/issues/27998

Description:
<!-- Don't request update of package. We have a script for that. https://alpha.de.repo.voidlinux.org/void-updates/void-updates.txt . However, a quality pull request may help. -->

### System

* xuname:  
  Void 5.9.16_1 x86_64 GenuineIntel uptodate rrFFFFFFF
* package:  
 spotify-1.1.42_2

### Expected behavior

Installing package works without any errors.

### Actual behavior

`post_extract()` calls `message_error()` because the checksum of Spotify's EULA does not match the current.

```bash
post_extract() {
	curl -L https://www.spotify.com/us/legal/end-user-agreement/ |
		w3m -dump -I utf-8 -T text/html |
		sed -n '/Spotify Terms and Conditions of Use/,/^USA$/p' > EULA

	filesum="$(xbps-digest EULA)"
	if [ "$filesum" != "$_license_checksum" ]; then
		msg_error "SHA256 mismatch for EULA:\n$filesum\n"
	fi
}
```

### Steps to reproduce the behavior

- enable restricted packages
- `./xbps-src pkg spotify` in `void-packages/`

### Questions

@teh-jazzman What is the reason behind checking the EULA and throwing an error?

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: spotify: SHA256 of EULA mismatching
  2021-01-17 23:24 [ISSUE] spotify: SHA256 of EULA mismatching waynevanson
                   ` (4 preceding siblings ...)
  2021-01-18 21:09 ` [ISSUE] [CLOSED] " Chocimier
@ 2021-10-08  5:23 ` Animeshz
  2021-10-09  7:06 ` teh-jazzman
  6 siblings, 0 replies; 8+ messages in thread
From: Animeshz @ 2021-10-08  5:23 UTC (permalink / raw)
  To: ml

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

New comment by Animeshz on void-packages repository

https://github.com/void-linux/void-packages/issues/27998#issuecomment-938354036

Comment:
License is changed again I guess?

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: spotify: SHA256 of EULA mismatching
  2021-01-17 23:24 [ISSUE] spotify: SHA256 of EULA mismatching waynevanson
                   ` (5 preceding siblings ...)
  2021-10-08  5:23 ` Animeshz
@ 2021-10-09  7:06 ` teh-jazzman
  6 siblings, 0 replies; 8+ messages in thread
From: teh-jazzman @ 2021-10-09  7:06 UTC (permalink / raw)
  To: ml

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

New comment by teh-jazzman on void-packages repository

https://github.com/void-linux/void-packages/issues/27998#issuecomment-939244045

Comment:
Yes, there have been substantial changes. I created #33439 but have not checked the new EULA for viability.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2021-10-09  7:06 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-17 23:24 [ISSUE] spotify: SHA256 of EULA mismatching waynevanson
2021-01-18 18:34 ` teh-jazzman
2021-01-18 19:00 ` ericonr
2021-01-18 19:03 ` ericonr
2021-01-18 21:09 ` Chocimier
2021-01-18 21:09 ` [ISSUE] [CLOSED] " Chocimier
2021-10-08  5:23 ` Animeshz
2021-10-09  7:06 ` teh-jazzman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).