Re: [arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-10-gddd508e

2017-11-04 Thread Luke Shumaker 
On Sat, 16 Sep 2017 19:35:21 -0400,
Jan Alexander Steffens wrote:
> On Sat, Sep 16, 2017 at 11:23 PM Luke Shumaker  wrote:
> 
>  On Thu, 14 Sep 2017 17:34:11 -0400,
>  Jan Steffens wrote:
>  > makechrootpkg: Reopen console to assign the CTTY
>  >
>  > nspawn does not give us a controlling terminal, hence we ignore
>  > interrupts. Apparently this was lost in systemd at some point.
>  >
>  > Hack around this by reopening the console to make it the controlling
>  > terminal.
> 
>  Does downgrading the chroot's glibc to 2.25 make this unnecessary? Or
>  are we seeing different issues?
> 
> No, this is indeed the same issue. I was mistaken when I thought this was 
> introduced in nspawn. 

FYI, I've reported this upstream and submitted patches.  Both the
appropriate committer and I hope to see this fixed in glibc 2.27.

https://sourceware.org/bugzilla/show_bug.cgi?id=22145
PATCH v1: https://sourceware.org/ml/libc-alpha/2017-10/msg00501.html
PATCH v2: https://sourceware.org/ml/libc-alpha/2017-11/msg00060.html

-- 
Happy hacking,
~ Luke Shumaker

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-13-g94160d6

2017-10-30 Thread Jan Steffens 
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The official devtools repository".

The branch, master has been updated
   via  94160d62b8d3de9d737f5ec70451fefa9ceda6d1 (commit)
   via  095e5305e45a32d4eee1e43a493200f4bc8455b3 (commit)
   via  e4db687d108d7e7cd32654b9c8eed4c7c12cb094 (commit)
  from  ddd508efc083fc9beb6f2c96e2537521b31c1e6f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
commit 94160d62b8d3de9d737f5ec70451fefa9ceda6d1
Author: Luke Shumaker 
Date:   Sun Sep 24 18:56:32 2017 -0400

use makepkg library instead of local function copies

This mirrors dbscripts commit
625fa02 by Pierre Schmitz  at 2017-04-18 14:20:49

commit 095e5305e45a32d4eee1e43a493200f4bc8455b3
Author: Luke Shumaker 
Date:   Sat Sep 16 18:00:11 2017 -0400

makechrootpkg: Fix function usage comments

A couple of the comments noting which globals are used by functions are
outdated/wrong.

 - download_sources() : Remove USER from the list.  It was always wrong.
   Originally, it should have been SUDO_USER (not USER), but I should have
   removed it entirely in 4f23609.

 - move_products() : Add SRCPKGDEST to the list.  Though the commit adding
   the comment was only recently upstreamed (as 2fd5931), it originated in
   2013 in a commit that has since  been rebased many times.  Anyway, in
   this rebasing, it missed move_products() starting to pay attention to
   SRCPKGDEST in fd1be1b (since nothing made git think there was a
   "conflict").

commit e4db687d108d7e7cd32654b9c8eed4c7c12cb094
Author: Luke Shumaker 
Date:   Sat Sep 16 17:52:39 2017 -0400

makechrootpkg: move init_variables() to be part of main()

The reason it wasn't moved before was just to keep the diffs
(with --ignore-all-space) smaller, to make merging and rebasing work
easier.  Moving code around in a file tends to make that difficult.

But, readability wise, it belongs in main().

---

Summary of changes:
 lib/common.sh| 100 ---
 makechrootpkg.in |  46 -
 2 files changed, 28 insertions(+), 118 deletions(-)


hooks/post-receive
-- 
The official devtools repository

Re: [arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-10-gddd508e

2017-09-16 Thread Jan Alexander Steffens via arch-projects 
On Sat, Sep 16, 2017 at 11:23 PM Luke Shumaker  wrote:

> On Thu, 14 Sep 2017 17:34:11 -0400,
> Jan Steffens wrote:
> > makechrootpkg: Reopen console to assign the CTTY
> >
> > nspawn does not give us a controlling terminal, hence we ignore
> > interrupts. Apparently this was lost in systemd at some point.
> >
> > Hack around this by reopening the console to make it the controlling
> > terminal.
>
> Does downgrading the chroot's glibc to 2.25 make this unnecessary?  Or
> are we seeing different issues?
>

No, this is indeed the same issue. I was mistaken when I thought this was
introduced in nspawn.

Re: [arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-10-gddd508e

2017-09-16 Thread Luke Shumaker 
On Thu, 14 Sep 2017 17:34:11 -0400,
Jan Steffens wrote:
> makechrootpkg: Reopen console to assign the CTTY
> 
> nspawn does not give us a controlling terminal, hence we ignore
> interrupts. Apparently this was lost in systemd at some point.
> 
> Hack around this by reopening the console to make it the controlling
> terminal.

Does downgrading the chroot's glibc to 2.25 make this unnecessary?  Or
are we seeing different issues?

  (You'll also need to downgrade binutils, as current binutils depends on 
glib>=2.26

   For convenience:
 
https://archive.archlinux.org/repos/2017/09/08/core/os/x86_64/glibc-2.25-7-x86_64.pkg.tar.xz
 
https://archive.archlinux.org/repos/2017/09/08/core/os/x86_64/binutils-2.28.0-4-x86_64.pkg.tar.xz
   )

-- 
Happy hacking,
~ Luke Shumaker

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-10-gddd508e

2017-09-14 Thread Jan Steffens 
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The official devtools repository".

The branch, master has been updated
   via  ddd508efc083fc9beb6f2c96e2537521b31c1e6f (commit)
   via  0f3778c3d36028e5c9d4e5ec702c8f167e1cebba (commit)
  from  6b55282bd702afd9a36409f85d9cb4c608ada5e5 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
commit ddd508efc083fc9beb6f2c96e2537521b31c1e6f
Author: Jan Alexander Steffens (heftig) 
Date:   Thu Sep 14 23:31:41 2017 +0200

makechrootpkg: Reopen console to assign the CTTY

nspawn does not give us a controlling terminal, hence we ignore
interrupts. Apparently this was lost in systemd at some point.

Hack around this by reopening the console to make it the controlling
terminal.

commit 0f3778c3d36028e5c9d4e5ec702c8f167e1cebba
Author: Jan Alexander Steffens (heftig) 
Date:   Thu Sep 14 22:21:47 2017 +0200

makechrootpkg: Prevent collecting coredumps

Coredumps from build chroots are not generally useful. Prevent
them from being generated.

Avoids a lot of annoyance from the GCC testsuite spawning lots of
systemd-coredump processes.

Just set the soft limit so the user can still raise it in the PKGBUILD
if they insist.

---

Summary of changes:
 makechrootpkg.in | 7 +++
 1 file changed, 7 insertions(+)


hooks/post-receive
-- 
The official devtools repository

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-8-g6b55282

2017-09-12 Thread Jan Steffens 
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The official devtools repository".

The branch, master has been updated
   via  6b55282bd702afd9a36409f85d9cb4c608ada5e5 (commit)
  from  6a7dcdeff9dee3c3b1192e61ad8e5a58f2215db0 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
commit 6b55282bd702afd9a36409f85d9cb4c608ada5e5
Author: Jan Alexander Steffens (heftig) 
Date:   Wed Sep 13 00:25:06 2017 +0200

Revert "arch-nspawn: Hack to give the inner process a controlling terminal"

Whoops, this will of course mess with nspawn arguments passed to
arch-nspawn.

---

Summary of changes:
 arch-nspawn.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
The official devtools repository

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-7-g6a7dcde

2017-09-12 Thread Jan Steffens 
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The official devtools repository".

The branch, master has been updated
   via  6a7dcdeff9dee3c3b1192e61ad8e5a58f2215db0 (commit)
   via  cd500d161fc174841486c6f204ae81d450086e92 (commit)
  from  5ebb9c16c055747e0501ac58b4cb05c20b8f7439 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
commit 6a7dcdeff9dee3c3b1192e61ad8e5a58f2215db0
Author: Jan Alexander Steffens (heftig) 
Date:   Wed Sep 13 00:08:30 2017 +0200

arch-nspawn: Hack to give the inner process a controlling terminal

This was lost at some point.

commit cd500d161fc174841486c6f204ae81d450086e92
Author: Jan Alexander Steffens (heftig) 
Date:   Tue Sep 12 23:26:34 2017 +0200

arch-nspawn: Pass --as-pid2

As not all commands we run are capable of reaping processes correctly.
For example, pacman is not.

---

Summary of changes:
 arch-nspawn.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
The official devtools repository

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-5-g5ebb9c1

2017-09-11 Thread Sébastien Luttringer 
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The official devtools repository".

The branch, master has been updated
   via  5ebb9c16c055747e0501ac58b4cb05c20b8f7439 (commit)
  from  fecf107c37062323cf77e08be7d2192e05cb6255 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
commit 5ebb9c16c055747e0501ac58b4cb05c20b8f7439
Author: Sébastien Luttringer 
Date:   Mon Sep 11 21:26:07 2017 +0200

Version 20170911

---

Summary of changes:
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
The official devtools repository

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-4-gfecf107

2017-09-11 Thread Sébastien Luttringer 
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The official devtools repository".

The branch, master has been updated
   via  fecf107c37062323cf77e08be7d2192e05cb6255 (commit)
  from  bd826752c9dc8f01917ee831302b6220ad09603a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
commit fecf107c37062323cf77e08be7d2192e05cb6255
Author: Sébastien Luttringer 
Date:   Wed Sep 6 23:27:50 2017 +0200

arch-nspawn: Force PATH to be the Arch default

systemd-nspawn use a default environ PATH value of:
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin

Since filesystem 2017.08, this is no more overrided by /etc/profile
to the Arch default:
PATH=/usr/local/sbin:/usr/local/bin:/usr/bin

---

Summary of changes:
 arch-nspawn.in | 1 +
 1 file changed, 1 insertion(+)


hooks/post-receive
-- 
The official devtools repository

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813-3-gbd82675

2017-08-24 Thread Jan Steffens 
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The official devtools repository".

The branch, master has been updated
   via  bd826752c9dc8f01917ee831302b6220ad09603a (commit)
   via  d0e684d2e998a9e7f241dc641c1f5273dba71036 (commit)
   via  000ea6c7bbf6191bd284468dde4eb352757e84df (commit)
  from  57151348843446141b90a4af365416d39bdcbbc2 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
commit bd826752c9dc8f01917ee831302b6220ad09603a
Author: Jan Alexander Steffens (heftig) 
Date:   Thu Aug 24 17:07:23 2017 +0200

makechrootpkg: Also look for -e as --noextract

commit d0e684d2e998a9e7f241dc641c1f5273dba71036
Author: Martchus 
Date:   Tue Aug 22 00:30:17 2017 +0200

makechrootpkg: Prevent removing build dir when --noextract specified

commit 000ea6c7bbf6191bd284468dde4eb352757e84df
Author: Jan Alexander Steffens (heftig) 
Date:   Thu Aug 24 17:03:32 2017 +0200

arch-nspawn: Add --keep-unit to prevent scope conflicts

---

Summary of changes:
 arch-nspawn.in   |  2 +-
 makechrootpkg.in | 22 +++---
 2 files changed, 12 insertions(+), 12 deletions(-)


hooks/post-receive
-- 
The official devtools repository

[arch-projects] [devtools] [GIT] The official devtools repository branch master updated. 20170813

2017-08-12 Thread Jan Steffens 
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The official devtools repository".

The branch, master has been updated
   via  57151348843446141b90a4af365416d39bdcbbc2 (commit)
  from  75fdff1811a0487f82c75b2e260da905102b4eea (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
commit 57151348843446141b90a4af365416d39bdcbbc2
Author: Jan Alexander Steffens (heftig) 
Date:   Sun Aug 13 01:24:11 2017 +0200

Version 20170813

---

Summary of changes:
 Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
The official devtools repository