Github messages for voidlinux
 help / color / mirror / Atom feed
* [ISSUE] Fixing patch whitespace
@ 2021-11-05 17:25 foopub
  2021-11-05 17:28 ` foopub
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: foopub @ 2021-11-05 17:25 UTC (permalink / raw)
  To: ml

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

New issue by foopub on void-packages repository

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

Description:
Regarding 7b4119d.
Seems reasonable to make a list of packages known to have bad patches and verify each build...

- [ ]  firefox-esr
- [ ] thundrebird



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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
@ 2021-11-05 17:28 ` foopub
  2021-11-05 17:47 ` q66
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: foopub @ 2021-11-05 17:28 UTC (permalink / raw)
  To: ml

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

New comment by foopub on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-962081798

Comment:
I actually get an error with the firefox-esr build, even tho there's no problem with the patching, so can't verify the build works: 

```
=> firefox-esr-78.14.0_1: running do_build ...
Traceback (most recent call last):
  File "/builddir/firefox-78.14.0/./mach", line 235, in <module>
    main(sys.argv[1:])
  File "/builddir/firefox-78.14.0/./mach", line 227, in main
    mach = get_mach()
  File "/builddir/firefox-78.14.0/./mach", line 209, in get_mach
    mach = check_and_get_mach(dir_path)
  File "/builddir/firefox-78.14.0/./mach", line 165, in check_and_get_mach
    return load_mach(dir_path, mach_path)
  File "/builddir/firefox-78.14.0/./mach", line 153, in load_mach
    return mach_bootstrap.bootstrap(dir_path)
  File "/builddir/firefox-78.14.0/build/mach_bootstrap.py", line 203, in bootstrap
    import mach.main
  File "/builddir/firefox-78.14.0/python/mach/mach/main.py", line 19, in <module>
    from collections import Iterable
ImportError: cannot import name 'Iterable' from 'collections' (/usr/lib/python3.10/collections/__init__.py)
=> ERROR: firefox-esr-78.14.0_1: do_build: './mach build' exited with 1
=> ERROR:   in do_build() at srcpkgs/firefox-esr/template:169
```

But `cat srcpkgs/firefox-esr/patches/* | grep py` shows no python files so so I guess this error isn't related to the patch? Is it safe check it off here? 



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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
  2021-11-05 17:28 ` foopub
@ 2021-11-05 17:47 ` q66
  2021-11-05 17:47 ` q66
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: q66 @ 2021-11-05 17:47 UTC (permalink / raw)
  To: ml

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

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-962094293

Comment:
that looks to be an unrelated error

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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
  2021-11-05 17:28 ` foopub
  2021-11-05 17:47 ` q66
@ 2021-11-05 17:47 ` q66
  2021-11-05 21:04 ` paper42
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: q66 @ 2021-11-05 17:47 UTC (permalink / raw)
  To: ml

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

New comment by q66 on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-962094293

Comment:
that looks to be an unrelated error (likely fallout from python 3.10 update)

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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
                   ` (2 preceding siblings ...)
  2021-11-05 17:47 ` q66
@ 2021-11-05 21:04 ` paper42
  2021-11-07 11:07 ` foopub
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: paper42 @ 2021-11-05 21:04 UTC (permalink / raw)
  To: ml

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

New comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-962219140

Comment:
It should be `from collections.abc import Iterable`, latest ESR is 91.3.0 which has this fixed. We can either update to it or make a simple patch for the current version.

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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
                   ` (3 preceding siblings ...)
  2021-11-05 21:04 ` paper42
@ 2021-11-07 11:07 ` foopub
  2021-11-07 17:26 ` paper42
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: foopub @ 2021-11-07 11:07 UTC (permalink / raw)
  To: ml

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

New comment by foopub on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-962590482

Comment:
> It should be `from collections.abc import Iterable`, latest ESR is 91.3.0 which has this fixed. We can either update to it or make a simple patch for the current version.

That's just the peak of the iceberg, there's too many errors to fix (I fixed 10 or so the other day before giving up) - better update to 91.3, or build with python 3.9.

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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
                   ` (4 preceding siblings ...)
  2021-11-07 11:07 ` foopub
@ 2021-11-07 17:26 ` paper42
  2021-11-09  8:43 ` newbluemoon
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: paper42 @ 2021-11-07 17:26 UTC (permalink / raw)
  To: ml

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

New comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-962649685

Comment:
> > It should be `from collections.abc import Iterable`, latest ESR is 91.3.0 which has this fixed. We can either update to it or make a simple patch for the current version.
> 
> That's just the peak of the iceberg, there's too many errors to fix (I fixed 10 or so the other day before giving up) - better update to 91.3

@Duncaen

> or build with python 3.9.

This is not possible, we are using python 3.10 everywhere.

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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
                   ` (5 preceding siblings ...)
  2021-11-07 17:26 ` paper42
@ 2021-11-09  8:43 ` newbluemoon
  2021-11-09 19:00 ` newbluemoon
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: newbluemoon @ 2021-11-09  8:43 UTC (permalink / raw)
  To: ml

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

New comment by newbluemoon on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-963930770

Comment:
Found another one in openjade, fixed in #33974.

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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
                   ` (6 preceding siblings ...)
  2021-11-09  8:43 ` newbluemoon
@ 2021-11-09 19:00 ` newbluemoon
  2022-06-11  2:13 ` github-actions
  2022-06-25  2:15 ` [ISSUE] [CLOSED] " github-actions
  9 siblings, 0 replies; 11+ messages in thread
From: newbluemoon @ 2021-11-09 19:00 UTC (permalink / raw)
  To: ml

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

New comment by newbluemoon on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-964444410

Comment:
pipewire is another one

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

* Re: Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
                   ` (7 preceding siblings ...)
  2021-11-09 19:00 ` newbluemoon
@ 2022-06-11  2:13 ` github-actions
  2022-06-25  2:15 ` [ISSUE] [CLOSED] " github-actions
  9 siblings, 0 replies; 11+ messages in thread
From: github-actions @ 2022-06-11  2:13 UTC (permalink / raw)
  To: ml

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

New comment by github-actions[bot] on void-packages repository

https://github.com/void-linux/void-packages/issues/33917#issuecomment-1152835726

Comment:
Issues become stale 90 days after last activity and are closed 14 days after that.  If this issue is still relevant bump it or assign it.

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

* Re: [ISSUE] [CLOSED] Fixing patch whitespace
  2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
                   ` (8 preceding siblings ...)
  2022-06-11  2:13 ` github-actions
@ 2022-06-25  2:15 ` github-actions
  9 siblings, 0 replies; 11+ messages in thread
From: github-actions @ 2022-06-25  2:15 UTC (permalink / raw)
  To: ml

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

Closed issue by foopub on void-packages repository

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

Description:
Regarding 7b4119d.
Seems reasonable to make a list of packages known to have bad patches and verify each build...

- [x]  firefox-esr
- [x] thundrebird



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

end of thread, other threads:[~2022-06-25  2:15 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-05 17:25 [ISSUE] Fixing patch whitespace foopub
2021-11-05 17:28 ` foopub
2021-11-05 17:47 ` q66
2021-11-05 17:47 ` q66
2021-11-05 21:04 ` paper42
2021-11-07 11:07 ` foopub
2021-11-07 17:26 ` paper42
2021-11-09  8:43 ` newbluemoon
2021-11-09 19:00 ` newbluemoon
2022-06-11  2:13 ` github-actions
2022-06-25  2:15 ` [ISSUE] [CLOSED] " github-actions

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).