[gentoo-dev] Automated Package Removal and Addition Tracker, for the week ending 2017-11-05 23:59 UTC

2017-11-05 Thread Robin H. Johnson
The attached list notes all of the packages that were added or removed
from the tree, for the week ending 2017-11-05 23:59 UTC.

Removals:
app-emulation/vmware-modules   20171101-13:45 dilfridge a2d54401ad1
app-emulation/vmware-player20171101-13:45 dilfridge a2d54401ad1
app-emulation/vmware-tools 20171101-13:45 dilfridge a2d54401ad1
app-emulation/vmware-workstation   20171101-13:45 dilfridge a2d54401ad1
app-office/qcharselect 20171030-01:33 asturm7daed1a859c
dev-perl/MARC-XML  20171101-00:27 kentnl3b8788d9c6f
dev-python/rope_py3k   20171102-11:49 mgorny59d7de54615
dev-util/gogland   20171105-20:38 mrueg 76fafcfe861
media-sound/lastfm-desktop 20171102-11:55 mgorny3f736b63ec6
net-misc/dnetstats 20171030-01:33 asturm7daed1a859c
sci-chemistry/mongochem20171030-01:33 asturm7daed1a859c
x11-libs/compizconfig-backend-kconfig4 20171105-10:11 asturmfb448622ef5

Additions:
app-vim/jedi   20171030-19:38 monsieurp e3751aa7bad
app-vim/pfsyntax   20171104-22:56 monsieurp ad1cc56d207
app-vim/vim2hs 20171105-23:34 monsieurp 3eb669ffb0a
dev-perl/Lab-VXI11 20171103-16:35 dilfridge a407fee0961
dev-perl/MARC-File-XML 20171101-00:23 kentnl293b8bcc1d2
dev-perl/Math-Utils20171101-15:06 kentnl9b36f4ad1dc
dev-perl/Net-DNS-Resolver-Mock 20171101-03:59 kentnl03cfff19dd3
dev-perl/USB-LibUSB20171103-16:41 dilfridge b7882bb6269
dev-perl/USB-TMC   20171103-16:47 dilfridge 7491b1c8fce
dev-php/maxmind-db-reader  20171105-22:52 mjo   80d52ea829a
dev-ruby/slow_enumerator_tools 20171105-05:50 graaffc18b19717f2
dev-util/gogland   20171030-17:42 mrueg b1445e00fcc
dev-util/goland20171103-14:46 mrueg 49eb881cc4b
sci-geosciences/opencpn-plugin-oesenc  20171030-22:21 mschiff   8736216753f
sys-fs/xwmfs   20171022-21:11 monsieurp d2ceae7ce68

--
Robin Hugh Johnson
Gentoo Linux Developer
E-Mail : robb...@gentoo.org
GnuPG FP   : 11AC BA4F 4778 E3F6 E4ED  F38E B27B 944E 3488 4E85
Removed Packages:
dev-util/gogland,removed,mrueg,20171105-20:38,76fafcfe861
x11-libs/compizconfig-backend-kconfig4,removed,asturm,20171105-10:11,fb448622ef5
media-sound/lastfm-desktop,removed,mgorny,20171102-11:55,3f736b63ec6
dev-python/rope_py3k,removed,mgorny,20171102-11:49,59d7de54615
app-emulation/vmware-modules,removed,dilfridge,20171101-13:45,a2d54401ad1
app-emulation/vmware-player,removed,dilfridge,20171101-13:45,a2d54401ad1
app-emulation/vmware-tools,removed,dilfridge,20171101-13:45,a2d54401ad1
app-emulation/vmware-workstation,removed,dilfridge,20171101-13:45,a2d54401ad1
dev-perl/MARC-XML,removed,kentnl,20171101-00:27,3b8788d9c6f
app-office/qcharselect,removed,asturm,20171030-01:33,7daed1a859c
net-misc/dnetstats,removed,asturm,20171030-01:33,7daed1a859c
sci-chemistry/mongochem,removed,asturm,20171030-01:33,7daed1a859c
Added Packages:
app-vim/vim2hs,added,monsieurp,20171105-23:34,3eb669ffb0a
sys-fs/xwmfs,added,monsieurp,20171022-21:11,d2ceae7ce68
dev-php/maxmind-db-reader,added,mjo,20171105-22:52,80d52ea829a
dev-ruby/slow_enumerator_tools,added,graaff,20171105-05:50,c18b19717f2
app-vim/pfsyntax,added,monsieurp,20171104-22:56,ad1cc56d207
dev-perl/USB-TMC,added,dilfridge,20171103-16:47,7491b1c8fce
dev-perl/USB-LibUSB,added,dilfridge,20171103-16:41,b7882bb6269
dev-perl/Lab-VXI11,added,dilfridge,20171103-16:35,a407fee0961
dev-util/goland,added,mrueg,20171103-14:46,49eb881cc4b
dev-perl/Math-Utils,added,kentnl,20171101-15:06,9b36f4ad1dc
dev-perl/Net-DNS-Resolver-Mock,added,kentnl,20171101-03:59,03cfff19dd3
dev-perl/MARC-File-XML,added,kentnl,20171101-00:23,293b8bcc1d2
app-vim/jedi,added,monsieurp,20171030-19:38,e3751aa7bad
sci-geosciences/opencpn-plugin-oesenc,added,mschiff,20171030-22:21,8736216753f
dev-util/gogland,added,mrueg,20171030-17:42,b1445e00fcc

Done.

Re: [gentoo-dev] [v1.0.3] GLEP 74: Full-tree verification using Manifest files

2017-11-05 Thread Michał Górny
W dniu czw, 02.11.2017 o godzinie 23∶43 +, użytkownik Robin H.
Johnson napisał:
> On Thu, Nov 02, 2017 at 08:11:59PM +0100, Michał Górny wrote:
> > Next version. Now without MISC/OPTIONAL, and with many clarifications.
> 
> Huge improvements in this version, I found it much easier to understand.
> 
> Nits: 
> - please stick to ASCII ellipsis. The unicode ellipsis is unreadable in
>   some monospace fonts.

Done. Also replaced '—' for consistency.

> 
> Further items inline:
> > Directory tree coverage
> > ---
> 
> ...
> > The file entries (except for ``IGNORE``) can be specified for regular
> > files only. Symbolic links are followed when opening files
> > and traversing directories. It is an error to specify an entry for
> > a different file type. If the tree contain files of other types
> > that are not otherwise ignored, they need to be covered by an explicit
> > ``IGNORE``.
> > 
> > All the local (non-``DIST``) files covered by a Manifest tree must
> > reside on the same filesystem. It is an error to specify entries
> > applying to files on another filesystem. If subdirectories
> > that are not otherwise ignored reside on a different filesystem, they
> > must be explicitly excluded via ``IGNORE``.
> 
> I would prefer this to say:
> 'If files that are not otherwise ignored reside on a different
> filesystem', as expanded from sub-directories.  
> This implicitly forbids following a symlink that crosses a filesystem
> boundary, and then matches the similar part of 'Tree layout
> restrictions'.

I've went for something even more explicit:

| If files or directories that are not otherwise ignored reside
| on a different filesystem, or symbolic links point to targets
| on a different filesystem, they must be explicitly excluded
| via ``IGNORE``.


> 
> > Rationale
> > =
> 
> ...
> > Tree layout restrictions
> > 
> > 
> > The algorithm is meant to work primarily with ebuild repositories which
> > normally contain only files and directories. Directories provide
> > no useful metadata for verification, and specifying special entries
> > for additional file types is purposeless. Therefore, the specification
> > is restricted to dealing with regular files.
> > 
> > The Gentoo repository does not use symbolic links. Some Gentoo
> > repositories do, however. To provide a simple solution for dealing with
> > symlinks without having to take care to implement special handling for
> > them, the common behavior of implicitly resolving them is used.
> > Therefore, symbolic links to files are stored as if they were regular
> > files, and symbolic links to directories are followed as if they were
> > regular directories.
> > 
> > Dotfiles are implicitly ignored as that is a common notion used
> > in software written for POSIX systems. All other common filenames
> > require explicit ``IGNORE`` lines.
> 
> 'common' in the second sentence seems odd. What about uncommon
> filenames? Maybe just s/other common filenames/other filenames/.

Done. The idea was to say 'do not put IGNORE for corner cases which are
better handled via PM config' but I guess it's not necessary here.

> 
> > An ability to inject additional ignore entries is provided to account
> > for site configuration affecting the repository tree — placing
> > additional files in it, skipping some of the categories from syncing.
> 
> Mention that the package manager may provide wildcards or regex in the
> additional entries. Eg: 'IGNORE **/metadata.xml' 

Done.

| This configuration can extend beyond the limits of this GLEP,
| e.g. by allowing wildcards or regular expressions.

> 
> > Non-strict Manifest verification
> > 
> 
> ...
> > The cases for stripping unnecessary files mostly focused around space
> > savings. For this purpose, stripping ``metadata.xml`` and similar files
> > has little value. It is much more common for users to strip whole
> > categories which can not be handled via the ``MISC`` type, and needs
> > a dedicated package manager mechanism. The same mechanism can also
> > handle files that used the ``MISC`` type.
> 
> Exclusion by package does happen as well. A list of categories or
> packages can be used for both the rsync exclusion and the IGNORE.

Rewritten to:

| It is much more common for users to strip whole packages
| or categories. The ``MISC`` type is not suitable for that,
| and so a dedicated package manager mechanism needs to be developed
| instead; possibly combining it with rsync exclusion list. The same
| mechanism can also handle files that historically used the ``MISC``
| type.

But it's merely a rationale, so I'd rather not spend another hour trying
to cover every corner case in it.

> 
> > Splitting distfile checksums from file checksums
> > 
> > 
> > Another problem with the current Manifest format is that the checksums
> > for fetched files are combined with checksums for local files
> > in a 

Re: [gentoo-dev] [PATCH 2/3] vim-plugin.eclass: use Portage internal variables.

2017-11-05 Thread Ulrich Mueller
> On Sun, 5 Nov 2017, Patrice Clement wrote:

> ACK. This code has been there for ages and I also don't think it makes sense 
> to
> keep it as it seems these operations are handled by Portage already. Mike
> (floppym) suggested to remove the whole if clause and call fperms instead:

> diff --git a/eclass/vim-plugin.eclass b/eclass/vim-plugin.eclass
> index abe9c7b3661..46908bb73e6 100644
> --- a/eclass/vim-plugin.eclass
> +++ b/eclass/vim-plugin.eclass
> @@ -35,19 +35,6 @@ vim-plugin_src_install() {
> has "${EAPI:-0}" 0 1 2 && ! use prefix && ED="${D}"
> local f

> -   if use !prefix && [[ ${EUID} -eq 0 ]] ; then
> -   ebegin "Fixing file permissions"
> -   # Make sure perms are good
> -   chmod -R a+rX "${S}" || die "chmod failed"
> -   find "${S}" -user  'portage' -exec chown root '{}' \; || die 
> "chown failed"
> -   if use userland_BSD || [[ ${CHOST} == *-darwin* ]] ; then
> -   find "${S}" -group 'portage' -exec chgrp wheel '{}' 
> \; || die "chgrp failed"
> -   else
> -   find "${S}" -group 'portage' -exec chgrp root '{}' \; 
> || die "chgrp failed"
> -   fi
> -   eend $?
> -   fi
> -
> # When globbing, if nothing exists, the shell literally returns the 
> glob
> # pattern. So turn on nullglob and extglob options to avoid this.
> eshopts_push -s extglob
> @@ -86,7 +73,7 @@ vim-plugin_src_install() {
> "couldn't move ${S} to ${ED}/usr/share/vim/vimfiles"

> # Fix remaining bad permissions
> -   chmod -R -x+X "${ED}"/usr/share/vim/vimfiles/ || die "chmod failed"
> +   fperms -R a+rX /usr/share/vim/vimfiles
>  }

LGTM (assuming that a+rX are the intended permissions).

Ulrich


pgpiKTrqOqLSE.pgp
Description: PGP signature


Re: [gentoo-dev] [PATCH 2/3] vim-plugin.eclass: use Portage internal variables.

2017-11-05 Thread Patrice Clement
Sunday 05 Nov 2017 00:26:33, Michał Górny wrote :
> W dniu sob, 04.11.2017 o godzinie 22∶48 +0100, użytkownik Patrice
> Clement napisał:
> > Closes: https://bugs.gentoo.org/469400
> > ---
> >  eclass/vim-plugin.eclass | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/eclass/vim-plugin.eclass b/eclass/vim-plugin.eclass
> > index a05cdb4520b..3c809768cf8 100644
> > --- a/eclass/vim-plugin.eclass
> > +++ b/eclass/vim-plugin.eclass
> > @@ -39,11 +39,11 @@ vim-plugin_src_install() {
> > ebegin "Fixing file permissions"
> > # Make sure perms are good
> > chmod -R a+rX "${S}" || die "chmod failed"
> > -   find "${S}" -user  'portage' -exec chown root '{}' \; || die 
> > "chown failed"
> > +   find "${S}" -user "${PORTAGE_USERNAME}" -exec chown root '{}' 
> > \; || die "chown failed"
> > if use userland_BSD || [[ ${CHOST} == *-darwin* ]] ; then
> > -   find "${S}" -group 'portage' -exec chgrp wheel '{}' \; 
> > || die "chgrp failed"
> > +   find "${S}" -group "${PORTAGE_GRPNAME}" -exec chgrp 
> > wheel '{}' \; || die "chgrp failed"
> > else
> > -   find "${S}" -group 'portage' -exec chgrp root '{}' \; 
> > || die "chgrp failed"
> > +   find "${S}" -group "${PORTAGE_GRPNAME}" -exec chgrp 
> > root '{}' \; || die "chgrp failed"
> > fi
> > eend $?
> > fi
> 
> This is going to die on every non-Portage package manager.
> 
> -- 
> Best regards,
> Michał Górny
> 
> 

Michal, Ulrich,

ACK. This code has been there for ages and I also don't think it makes sense to
keep it as it seems these operations are handled by Portage already. Mike
(floppym) suggested to remove the whole if clause and call fperms instead:

diff --git a/eclass/vim-plugin.eclass b/eclass/vim-plugin.eclass
index abe9c7b3661..46908bb73e6 100644
--- a/eclass/vim-plugin.eclass
+++ b/eclass/vim-plugin.eclass
@@ -35,19 +35,6 @@ vim-plugin_src_install() {
has "${EAPI:-0}" 0 1 2 && ! use prefix && ED="${D}"
local f

-   if use !prefix && [[ ${EUID} -eq 0 ]] ; then
-   ebegin "Fixing file permissions"
-   # Make sure perms are good
-   chmod -R a+rX "${S}" || die "chmod failed"
-   find "${S}" -user  'portage' -exec chown root '{}' \; || die 
"chown failed"
-   if use userland_BSD || [[ ${CHOST} == *-darwin* ]] ; then
-   find "${S}" -group 'portage' -exec chgrp wheel '{}' \; 
|| die "chgrp failed"
-   else
-   find "${S}" -group 'portage' -exec chgrp root '{}' \; 
|| die "chgrp failed"
-   fi
-   eend $?
-   fi
-
# When globbing, if nothing exists, the shell literally returns the glob
# pattern. So turn on nullglob and extglob options to avoid this.
eshopts_push -s extglob
@@ -86,7 +73,7 @@ vim-plugin_src_install() {
"couldn't move ${S} to ${ED}/usr/share/vim/vimfiles"

# Fix remaining bad permissions
-   chmod -R -x+X "${ED}"/usr/share/vim/vimfiles/ || die "chmod failed"
+   fperms -R a+rX /usr/share/vim/vimfiles
 }

Please let me know what you guys think.

-- 
Patrice Clement
Gentoo Linux developer
http://www.gentoo.org