[ISSUE] Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[ISSUE] Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[4ricci]

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

New issue by 4ricci on void-packages repository

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

Description:
`perl-HTTP-Message` is a check dependency of `perl-HTML-Parser`,
`perl-HTML-Parser` is a runtime and build dependency of `perl-HTTP-Message`.

`./xbps-src -Q pkg perl-HTTP-Message` returns the error:
```
=> ERROR: Build-time cyclic dependency perl-HTML-Parser(host), perl-HTTP-Message(host), perl-HTML-Parser(host) detected.
```

`./xbps-src -Q pkg perl-HTML-Parser` returns the error:
```
=> ERROR: Build-time cyclic dependency perl-HTTP-Message(host), perl-HTML-Parser(host), perl-HTTP-Message(host) detected.
```

Re: Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[ericonr]

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/issues/32468#issuecomment-897723363

Comment:
I think the correct path forward would be disabling tests for `perl-HTML-Parser` (or see if they can run without `perl-HTTP-Message`. Otherwise this means bootstrap is broken when tests are enabled, and we want to make bootstrap with tests possible.

Re: Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[Chocimier]

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

New comment by Chocimier on void-packages repository

https://github.com/void-linux/void-packages/issues/32468#issuecomment-897744667

Comment:
On the contrary, I think cycles due to checkdepends are better than not testing package at all. In case like here, one can first build package(s) locally without tests, than rebuild with tests enabled.

Re: Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[dkwo]

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

New comment by dkwo on void-packages repository

https://github.com/void-linux/void-packages/issues/32468#issuecomment-898940373

Comment:
I also had to deal with the same issue here https://github.com/void-linux/void-packages/pull/32229

Re: [ISSUE] [CLOSED] Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[4ricci]

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

Closed issue by 4ricci on void-packages repository

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

Description:
`perl-HTTP-Message` is a check dependency of `perl-HTML-Parser`,
`perl-HTML-Parser` is a runtime and build dependency of `perl-HTTP-Message`.

When updating both packages (<https://github.com/4ricci/void-packages/pull/31>), `./xbps-src -Q pkg perl-HTTP-Message` returns the error:
```
=> ERROR: Build-time cyclic dependency perl-HTML-Parser(host), perl-HTTP-Message(host), perl-HTML-Parser(host) detected.
```

`./xbps-src -Q pkg perl-HTML-Parser` returns the error:
```
=> ERROR: Build-time cyclic dependency perl-HTTP-Message(host), perl-HTML-Parser(host), perl-HTTP-Message(host) detected.
```

Re: Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[q66]

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

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/issues/32468#issuecomment-922357937

Comment:
i think not testing is better because if tests are not cyclic, they can be properly automated (we could even have the builders run them eventually) while if you allow cycles they can't (and tests on stuff like this are mostly meaningless either way, i see no reason to let a few rare cases break it)

Re: Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[q66]

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

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/issues/32468#issuecomment-922358093

Comment:
also i think checkdepends are in general a poor idea because it means that your build environment differs if you're running tests vs if you're not

while ideally the build environment should only ever be affected by build options

Re: Cyclic dependency: perl-HTML-Parser <-> perl-HTTP-Message

[dkwo]

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

New comment by dkwo on void-packages repository

https://github.com/void-linux/void-packages/issues/32468#issuecomment-922448457

Comment:
Then, shall we treat checkdepends just as (host)makedepends, regardless of whether we are permorming tests or not?