Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 116984 - email addresses with multiple "@" not supported by mod_mail elog module
Summary: email addresses with multiple "@" not supported by mod_mail elog module
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Core - Interface (emerge) (show other bugs)
Hardware: x86 Linux
: High major
Assignee: Portage team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-28 07:01 UTC by Jan-Fabian Humann
Modified: 2007-01-11 04:52 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
a revised mod_mail.py in order to get sending via GMX working... (mod_mail.patch,4.17 KB, patch)
2005-12-28 09:46 UTC, Jan-Fabian Humann
Details | Diff
now with correct indentation (second.patch,3.59 KB, patch)
2005-12-30 13:45 UTC, Jan-Fabian Humann
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jan-Fabian Humann 2005-12-28 07:01:39 UTC
I just updated my portage and then tried to emerge nvidia-glx, but right at the end of the emerge it gives me this:


--- !empty dir /usr/lib/opengl/nvidia/lib
--- !empty dir /usr/lib/opengl/nvidia/include
--- !empty dir /usr/lib/opengl/nvidia/extensions
--- !empty dir /usr/lib/opengl/nvidia
--- !empty dir /usr/lib/opengl
--- !empty dir /usr/lib/modules/drivers
--- !empty dir /usr/lib/modules
--- !empty dir /usr/lib
--- !empty dir /usr/bin
--- !empty dir /usr
Switching to xorg-x11 OpenGL interface... done
>>> original instance of package unmerged safely.
Switching to nvidia OpenGL interface... done

 * To use the Nvidia GLX, run "eselect opengl set nvidia"

 * You may also be interested in media-video/nvidia-settings

 * nVidia have requested that any bug reports submitted have the
 * output of /usr/bin/nvidia-bug-report.sh included.
>>> Regenerating /etc/ld.so.cache...
>>> media-video/nvidia-glx-1.0.8178 merged.
Traceback (most recent call last):
  File "/usr/bin/emerge", line 3335, in ?
    mydepgraph.merge(pkglist)
  File "/usr/bin/emerge", line 2037, in merge
    retval=portage.doebuild(y,"merge",myroot,self.pkgsettings,edebug,tree="porttree")
  File "/usr/lib/portage/pym/portage.py", line 2858, in doebuild
    return merge(mysettings["CATEGORY"],mysettings["PF"],mysettings["D"],mysettings["BUILDDIR"]+"/build-info",myroot,mysettings,myebuild=mysettings["EBUILD"],mytree=tree)
  File "/usr/lib/portage/pym/portage.py", line 3033, in merge
    return mylink.merge(pkgloc,infloc,myroot,myebuild)
  File "/usr/lib/portage/pym/portage.py", line 6385, in merge
    return self.treewalk(mergeroot,myroot,inforoot,myebuild,cleanup=cleanup)
  File "/usr/lib/portage/pym/portage.py", line 6086, in treewalk
    elog_process(self.mycpv, self.settings)
  File "/usr/lib/portage/pym/portage.py", line 504, in elog_process
    m.process(mysettings, cpv, mylogentries, fulllog)
  File "/usr/lib/portage/pym/elog_modules/mod_mail.py", line 21, in process
    myauthdata, myconndata = mymailuri.split("@")
ValueError: too many values to unpack


After that I tried to just emerge -pvD --update world, and well, here's the output:

>>> emerge (2 of 56) sys-devel/gnuconfig-20051113 to /
>>> Downloading ftp://ftp.tu-clausthal.de/pub/linux/gentoo/distfiles/gnuconfig-20051113.tar.bz2
--15:46:02--  ftp://ftp.tu-clausthal.de/pub/linux/gentoo/distfiles/gnuconfig-20051113.tar.bz2
           => `/usr/portage/distfiles/gnuconfig-20051113.tar.bz2'
Aufl
Comment 1 Jan-Fabian Humann 2005-12-28 07:01:39 UTC
I just updated my portage and then tried to emerge nvidia-glx, but right at the end of the emerge it gives me this:


--- !empty dir /usr/lib/opengl/nvidia/lib
--- !empty dir /usr/lib/opengl/nvidia/include
--- !empty dir /usr/lib/opengl/nvidia/extensions
--- !empty dir /usr/lib/opengl/nvidia
--- !empty dir /usr/lib/opengl
--- !empty dir /usr/lib/modules/drivers
--- !empty dir /usr/lib/modules
--- !empty dir /usr/lib
--- !empty dir /usr/bin
--- !empty dir /usr
Switching to xorg-x11 OpenGL interface... done
>>> original instance of package unmerged safely.
Switching to nvidia OpenGL interface... done

 * To use the Nvidia GLX, run "eselect opengl set nvidia"

 * You may also be interested in media-video/nvidia-settings

 * nVidia have requested that any bug reports submitted have the
 * output of /usr/bin/nvidia-bug-report.sh included.
>>> Regenerating /etc/ld.so.cache...
>>> media-video/nvidia-glx-1.0.8178 merged.
Traceback (most recent call last):
  File "/usr/bin/emerge", line 3335, in ?
    mydepgraph.merge(pkglist)
  File "/usr/bin/emerge", line 2037, in merge
    retval=portage.doebuild(y,"merge",myroot,self.pkgsettings,edebug,tree="porttree")
  File "/usr/lib/portage/pym/portage.py", line 2858, in doebuild
    return merge(mysettings["CATEGORY"],mysettings["PF"],mysettings["D"],mysettings["BUILDDIR"]+"/build-info",myroot,mysettings,myebuild=mysettings["EBUILD"],mytree=tree)
  File "/usr/lib/portage/pym/portage.py", line 3033, in merge
    return mylink.merge(pkgloc,infloc,myroot,myebuild)
  File "/usr/lib/portage/pym/portage.py", line 6385, in merge
    return self.treewalk(mergeroot,myroot,inforoot,myebuild,cleanup=cleanup)
  File "/usr/lib/portage/pym/portage.py", line 6086, in treewalk
    elog_process(self.mycpv, self.settings)
  File "/usr/lib/portage/pym/portage.py", line 504, in elog_process
    m.process(mysettings, cpv, mylogentries, fulllog)
  File "/usr/lib/portage/pym/elog_modules/mod_mail.py", line 21, in process
    myauthdata, myconndata = mymailuri.split("@")
ValueError: too many values to unpack


After that I tried to just emerge -pvD --update world, and well, here's the output:

>>> emerge (2 of 56) sys-devel/gnuconfig-20051113 to /
>>> Downloading ftp://ftp.tu-clausthal.de/pub/linux/gentoo/distfiles/gnuconfig-20051113.tar.bz2
--15:46:02--  ftp://ftp.tu-clausthal.de/pub/linux/gentoo/distfiles/gnuconfig-20051113.tar.bz2
           => `/usr/portage/distfiles/gnuconfig-20051113.tar.bz2'
Auflösen des Hostnamen »ftp.tu-clausthal.de«.... 139.174.2.36
Verbindungsaufbau zu ftp.tu-clausthal.de|139.174.2.36|:21... verbunden.
Anmelden als anonymous ... Angemeldet!
==> SYST ... fertig.    ==> PWD ... fertig.
==> TYPE I ... fertig.  ==> CWD /pub/linux/gentoo/distfiles ... fertig.
==> PASV ... fertig.    ==> RETR gnuconfig-20051113.tar.bz2 ... fertig.
Länge: 38.144 (37K) (unmaßgeblich)

100%[===================================>] 38.144        70.43K/s             

15:46:03 (70.15 KB/s) - »/usr/portage/distfiles/gnuconfig-20051113.tar.bz2« gespeichert [38144]

>>> checksums files   ;-) gnuconfig-20051113.ebuild
>>> checksums files   ;-) gnuconfig-20050602.ebuild
>>> checksums files   ;-) files/digest-gnuconfig-20050602
>>> checksums files   ;-) files/digest-gnuconfig-20051113
>>> checksums src_uri ;-) gnuconfig-20051113.tar.bz2
>>> Unpacking source...
>>> Unpacking gnuconfig-20051113.tar.bz2 to /var/tmp/portage/gnuconfig-20051113/work
 * Applying config.guess.uclibc.patch ...                                [ ok ]
 * Applying config.sub.ps2.patch ...                                     [ ok ]
 * Applying config.sub.sh.patch ...                                      [ ok ]
>>> Source unpacked.
>>> Compiling source in /var/tmp/portage/gnuconfig-20051113/work ...
>>> Source compiled.
>>> Test phase [not enabled]: sys-devel/gnuconfig-20051113

>>> Install gnuconfig-20051113 into /var/tmp/portage/gnuconfig-20051113/image/ category sys-devel
man:
prepallstrip:
>>> Completed installing gnuconfig-20051113 into /var/tmp/portage/gnuconfig-20051113/image/

>>> Merging sys-devel/gnuconfig-20051113 to /
--- /usr/
--- /usr/share/
--- /usr/share/gnuconfig/
>>> /usr/share/gnuconfig/config.sub
>>> /usr/share/gnuconfig/config.guess
--- /usr/share/doc/
>>> /usr/share/doc/gnuconfig-20051113/
>>> /usr/share/doc/gnuconfig-20051113/ChangeLog.gz
>>> Regenerating /etc/ld.so.cache...
>>> sys-devel/gnuconfig-20051113 merged.
Traceback (most recent call last):
  File "/usr/bin/emerge", line 3335, in ?
    mydepgraph.merge(pkglist)
  File "/usr/bin/emerge", line 2037, in merge
    retval=portage.doebuild(y,"merge",myroot,self.pkgsettings,edebug,tree="porttree")
  File "/usr/lib/portage/pym/portage.py", line 2858, in doebuild
    return merge(mysettings["CATEGORY"],mysettings["PF"],mysettings["D"],mysettings["BUILDDIR"]+"/build-info",myroot,mysettings,myebuild=mysettings["EBUILD"],mytree=tree)
  File "/usr/lib/portage/pym/portage.py", line 3033, in merge
    return mylink.merge(pkgloc,infloc,myroot,myebuild)
  File "/usr/lib/portage/pym/portage.py", line 6385, in merge
    return self.treewalk(mergeroot,myroot,inforoot,myebuild,cleanup=cleanup)
  File "/usr/lib/portage/pym/portage.py", line 6086, in treewalk
    elog_process(self.mycpv, self.settings)
  File "/usr/lib/portage/pym/portage.py", line 504, in elog_process
    m.process(mysettings, cpv, mylogentries, fulllog)
  File "/usr/lib/portage/pym/elog_modules/mod_mail.py", line 21, in process
    myauthdata, myconndata = mymailuri.split("@")
ValueError: too many values to unpack
Comment 2 Jakub Moc (RETIRED) gentoo-dev 2005-12-28 07:09:54 UTC
emerge --info, please...
Comment 3 Jan-Fabian Humann 2005-12-28 07:11:34 UTC
Portage 2.1_pre2 (default-linux/x86/2005.1, gcc-3.3.5-20050130, glibc-2.3.4.20041102-r1, 2.6.14-gentoo-r5 i686)
=================================================================
System uname: 2.6.14-gentoo-r5 i686 Intel(R) Pentium(R) 4 CPU 3.06GHz
Gentoo Base System version 1.6.13
dev-lang/python:     2.3.5-r2, 2.4.2
sys-apps/sandbox:    1.2.17
sys-devel/autoconf:  2.13, 2.59-r6
sys-devel/automake:  1.4_p6, 1.5, 1.6.3, 1.7.9-r1, 1.8.5-r3, 1.9.5
sys-devel/binutils:  2.15.92.0.2-r10, 2.16.1-r1
sys-devel/libtool:   1.5.18-r1
virtual/os-headers:  2.6.11-r2
ACCEPT_KEYWORDS="x86 ~x86"
AUTOCLEAN="yes"
CBUILD="i686-pc-linux-gnu"
CFLAGS="-O3 -march=pentium4 -pipe"
CHOST="i686-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/kde/2/share/config /usr/kde/3/share/config /usr/lib/X11/xkb /usr/share/config /var/qmail/control"
CONFIG_PROTECT_MASK="/etc/gconf /etc/terminfo /etc/env.d"
CXXFLAGS="-O3 -march=pentium4 -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoconfig distlocks sandbox sfperms strict"
GENTOO_MIRRORS="ftp://ftp.tu-clausthal.de/pub/linux/gentoo/ ftp://sunsite.informatik.rwth-aachen.de/pub/Linux/gentoo http://linux.rz.ruhr-uni-bochum.de/download/gentoo-mirror/ ftp://linux.rz.ruhr-uni-bochum.de/gentoo-mirror/ http://ftp.uni-erlangen.de/pub/mirrors/gentoo ftp://ftp.uni-erlangen.de/pub/mirrors/gentoo ftp://ftp.join.uni-muenster.de/pub/linux/distributions/gentoo ftp://ftp.wh2.tu-dresden.de/pub/mirrors/gentoo ftp://ftp.join.uni-muenster.de/pub/linux/distributions/gentoo ftp://ftp6.uni-muenster.de/pub/linux/distributions/gentoo http://mirrors.sec.informatik.tu-darmstadt.de/gentoo/ http://ftp-stud.fht-esslingen.de/pub/Mirrors/gentoo/ ftp://ftp-stud.fht-esslingen.de/pub/Mirrors/gentoo/ ftp://ftp.gentoo.mesh-solutions.com/gentoo/ http://pandemonium.tiscali.de/pub/gentoo/ ftp://pandemonium.tiscali.de/pub/gentoo/ ftp://ftp.rz.tu-bs.de/pub/mirror/ftp.gentoo.org/gentoo-distfiles/ http://gentoo.intergenia.de"
LANG="de_DE@euro"
LC_ALL="de_DE@euro"
LINGUAS="de en"
MAKEOPTS="-j3"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
SYNC="rsync://rsync.de.gentoo.org/gentoo-portage"
USE="x86 X a52 aac aalib alsa apm audiofile avi bash-completion berkdb bitmap-fonts bmp bzip2 bzlib cdr cjk crypt curl divx4linux doc dvd dvdr dvdread eds emacs emboss encode exif expat fam ffmpeg flac flash foomaticdb fortran ftp gd gdbm gif gnome gphoto2 gpm gstreamer gtk gtk2 iconv icq ieee1394 imagemagick imap imlib ipv6 jabber java javascript jpeg lcms ldap libg++ libwww lirc lm_sensors lua mad maildir mailwrapper mbox mikmod mime mmx mng mono motif mozilla mp3 mpeg mysql ncurses nls nvidia ogg oggvorbis opengl oss pam pcre pdflib perl php png python qt quicktime readline ruby samba sdl sockets spell sqlite sse ssl svg tcpd tiff truetype truetype-fonts type1-fonts udev unicode usb v4l vcd vorbis win32codecs wmf wxwindows xine xml2 xmms xv xvid zlib elibc_glibc kernel_linux linguas_de linguas_en userland_GNU"
Unset:  ASFLAGS, CTARGET, LDFLAGS, PORTDIR_OVERLAY
Comment 4 Jan-Fabian Humann 2005-12-28 09:18:30 UTC
Ah, just looked at the output and saw it ahs something to do with this new ELOG System, therefore my make.conf elog variables:



PORTAGE_ELOG_CLASSES="warn error log info"
PORTAGE_ELOG_SYSTEM="save mail"
PORTAGE_ELOG_MAILURI="malastare@gmx.net malastare@gmx.net:mypassword@smtp.gmx.net"

And I guess that's exactly the problem: 

File "/usr/lib/portage/pym/elog_modules/mod_mail.py", line 21, in process
    myauthdata, myconndata = mymailuri.split("@")
ValueError: too many values to unpack

This happens because there are two @'s in my MAILURI setting, but I actually need em both, because GMX, my mail provider requests the complete mailadress as username for its smtp services, the second one is used for the server..so you maybe should just change the layout of that setting, to allow usernames which include an @...
Comment 5 Jan-Fabian Humann 2005-12-28 09:46:27 UTC
Created attachment 75678 [details, diff]
a revised mod_mail.py in order to get sending via GMX working...

Yep, I fucked your indenting up :P
Comment 6 Jan-Fabian Humann 2005-12-28 09:52:36 UTC
Ah, did think bugzilla would not delete the comment I wrote before submitting the patch -.-

So In order to get this fixed for me I rewrote a bit of mod_mail.py, first major change: due to the differenciated settings for elog_mail in make.conf it is now possible to use @ in username (Which is important for GMX users), second addition is the FROM setting, which is as well needed for example for GMX, which will request a fully quallified mail adress as FROM argument, also when logging in, 'portage' is simply not enough.
I hope some portage dev can implement something prettier than this into portage so  I can use GMX for sending ;)
Comment 7 Jason Stubbs (RETIRED) gentoo-dev 2005-12-29 22:47:02 UTC
A patch that uses tabs instead of spaces would be nice. That patch doesn't show what has changed.
Comment 8 Jan-Fabian Humann 2005-12-30 13:45:11 UTC
Created attachment 75819 [details, diff]
now with correct indentation

Alright, this time didn't brak the indentation...
Comment 9 Brian Harring (RETIRED) gentoo-dev 2006-02-05 03:57:27 UTC
Marius, your area of code- comments on this?
Comment 10 Alexander Skwar 2006-02-19 06:04:21 UTC
(In reply to comment #3)

> This happens because there are two @'s in my MAILURI setting, but I actually
> need em both, because GMX, my mail provider requests the complete mailadress as
> username for its smtp services,

Actually, that's not true. With GMX, you can also use your Kundennummer.

> maybe should just change the layout of that setting, to allow usernames which
> include an @...

Or maybe allow something like "%40"? 0x40 is, in iso-8859 compatible charsets, equal to @.
Comment 11 Marius Mauch (RETIRED) gentoo-dev 2007-01-11 04:52:29 UTC
This should be fixed since 2.1.1