if a manifest is missing (just rm Manifest to reproduce), you get this error: !!! Missing digest for None what exactly is "None" ? also, the latest version is a huge pita for developers for doing unpacks. every file in an ebuild has to have a correct manifest in order to do 'ebuild ... unpack'. previously, missing manifests were ignored when executing ebuild. for people doing patch tarball testing, this gets to be unusable.
It should work if you insert "digest" just before "unpack" in you command line. If that works then we can make it the default behavior to call "digest" first.
i'm not sure how that's any better from doing nothing at all ?
Does FEATURES=-strict help? I guess we could make something like that default for the ebuild command.
Created attachment 155885 [details, diff] Allow digestcheck() to succeed with -strict and missing Manifest This should fix it work with FEATURES=-strict.
i'm already running -strict ;) Portage 2.2_pre7-r1 (default-linux/amd64/2006.1/desktop, gcc-4.3.0, glibc-2.8_p20080602-r0, 2.6.25 x86_64) ================================================================= System uname: 2.6.25 x86_64 AMD Athlon(tm) 64 X2 Dual Core Processor 4200+ Timestamp of tree: Sun, 08 Jun 2008 04:00:01 +0000 distcc 2.18.3 x86_64-pc-linux-gnu (protocols 1 and 2) (default port 3632) [disabled] ccache version 2.4 [enabled] app-shells/bash: 3.2_p39 dev-lang/python: 2.4.4-r8, 2.5.2-r2 dev-python/pycrypto: 2.0.1-r6 dev-util/ccache: 2.4-r7 sys-apps/baselayout: 2.0.0 sys-apps/openrc: 9999 sys-apps/sandbox: 1.2.18.1-r2 sys-devel/autoconf: 2.13, 2.62 sys-devel/automake: 1.4_p6, 1.5, 1.6.3, 1.7.9-r1, 1.8.5-r3, 1.9.6-r2, 1.10.1-r1 sys-devel/binutils: 2.15.92.0.2-r10, 2.16-r1, 2.16.1, 2.16.1-r3, 2.16.90.0.3, 2.16.91.0.1, 2.16.91.0.2, 2.16.91.0.3, 2.16.91.0.4, 2.16.91.0.5, 2.16.91.0.6, 2.16.91.0.7, 2.16.92, 2.16.93, 2.16.94, 2.17-r1, 2.17.50.0.2, 2.17.50.0.3, 2.17.50.0.4, 2.17.50.0.5, 2.17.50.0.6, 2.17.50.0.7, 2.17.50.0.8, 2.17.50.0.9, 2.17.50.0.10, 2.17.50.0.11, 2.17.50.0.12, 2.17.50.0.13, 2.17.50.0.14, 2.17.50.0.15, 2.17.50.0.16, 2.17.50.0.17, 2.17.50.0.18, 2.18-r1, 2.18.50.0.1, 2.18.50.0.2, 2.18.50.0.3, 2.18.50.0.4, 2.18.50.0.5, 2.18.50.0.6, 2.18.50.0.7 sys-devel/gcc-config: 1.4.1 sys-devel/libtool: 2.2.4 virtual/os-headers: 2.6.25-r3 ACCEPT_KEYWORDS="amd64 ~amd64" CBUILD="x86_64-pc-linux-gnu" CFLAGS="-O2 -march=k8 -pipe -g -Wimplicit-function-declaration" CHOST="x86_64-pc-linux-gnu" CONFIG_PROTECT="/etc /usr/kde/3.5/env /usr/kde/3.5/share/config /usr/kde/3.5/shutdown /usr/share/config /var/qmail/alias /var/qmail/control /var/vpopmail/etc" CONFIG_PROTECT_MASK="/etc/env.d /etc/fonts/fonts.conf /etc/gconf /etc/gentoo-release /etc/php/apache2-php5/ext-active/ /etc/php/cgi-php5/ext-active/ /etc/php/cli-php5/ext-active/ /etc/revdep-rebuild /etc/terminfo /etc/texmf/web2c /etc/udev/rules.d /var/lib/rpm" CXXFLAGS="-O2 -march=k8 -pipe -g" DISTDIR="/usr/portage/distfiles" FEATURES="assume-digests buildsyspkg ccache collision-protect cvs distlocks noinfo parallel-fetch preserve-libs sandbox sfperms sign splitdebug unmerge-orphans userfetch" GENTOO_MIRRORS="http://distfiles.gentoo.org http://distro.ibiblio.org/pub/linux/distributions/gentoo" INSTALL_MASK="/usr/share/applications /usr/kde/*/share/applications" LANG="en_US.UTF8" LDFLAGS="-Wl,-O1 -Wl,-z,relro" LINGUAS="en en_GB de es cs ru" MAKEOPTS="-j6" PKGDIR="/usr/portage/packages" PORTAGE_COMPRESS="lzma" PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --compress --force --whole-file --delete --stats --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages" PORTAGE_TMPDIR="/var/tmp" PORTDIR="/usr/portage" PORTDIR_OVERLAY="/usr/local/portage /usr/local/src/gentoo/overlays/vapier/enlightenment" SYNC="rsync://gentoo/gentoo-portage" USE="X a52 aac aalib accessibility acl adns agg aio alsa amd64 apache2 asf audiofile berkdb bitmap-fonts bzip2 cairo cddb cdparanoia cdr cli console cracklib crypt cups curl dba directfb divx4linux dri dts dvb dvd dvdr dvdread emboss encode exif fbcon ffmpeg firefox flac flash fortran ftp gcj gd gif glitz glut gmp gphoto2 gpm gtk gtk2 iconv imap imlib ipv6 isdnlog jbig joystick jpeg jpeg2k libcaca libedit lzo lzw mad maildir matroska midi mikmod mime mjpeg mng modplug mp3 mpeg mplayer mudflap multislot musepack mysql ncurses nls nptl nptlonly nsplugin nvidia objc objc-gc offensive ogg oggvorbis openal opengl openmp pcre pdf perl pic png ppds pppd pulseaudio python qt3 qt4 quicktime readline reflection samba sdl session sndfile snmp spell spl sqlite ssl subtitles svg syslog tcl tcltk tcpd tga theora threads tiff tk truetype truetype-fonts type1-fonts unicode usb vcd vorbis wma wmf xanim xattr xcomposite xine xinerama xml xml2 xorg xpm xrandr xv xvid xvmc zip zlib" ALSA_CARDS="ali5451 als4000 atiixp atiixp-modem bt87x ca0106 cmipci emu10k1x ens1370 ens1371 es1938 es1968 fm801 hda-intel intel8x0 intel8x0m maestro3 trident usb-audio via82xx via82xx-modem ymfpci" ALSA_PCM_PLUGINS="adpcm alaw asym copy dmix dshare dsnoop empty extplug file hooks iec958 ioplug ladspa lfloat linear meter mulaw multi null plug rate route share shm softvol" APACHE2_MODULES="actions alias auth_basic authn_alias authn_anon authn_dbm authn_default authn_file authz_dbm authz_default authz_groupfile authz_host authz_owner authz_user autoindex cache dav dav_fs dav_lock deflate dir disk_cache env expires ext_filter file_cache filter headers include info log_config logio mem_cache mime mime_magic negotiation rewrite setenvif speling status unique_id userdir usertrack vhost_alias" CAMERAS="canon" ELIBC="glibc" INPUT_DEVICES="mouse keyboard joystick void" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LINGUAS="en en_GB de es cs ru" USERLAND="GNU" VIDEO_CARDS="nvidia nv ati r128 radeon vga sisusb" Unset: CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, LC_ALL, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS
Does the patch help though? It seems to help for me with -strict when the manifest is missing.
patch doesnt help. i still get the missing error.
Well, it seems to me that the patch should help in the case of a missing Manifest with FEATURES=-strict, so I'm not sure what I'm missing. If you want, you can reproduce the problem with --debug and FEATURES=python-trace enabled. That produces an extremely verbose log, but the most interesting part should be in the tail 2000 lines or so.
just test like this: ebuild portage-2.2_pre7-r1.ebuild clean unpack compile install qmerge echo > Manifest ebuild portage-2.2_pre7-r1.ebuild clean unpack compile install qmerge the second one will fail
(of course, i applied the strict.patch to /usr/lib/portage/pym/portage/__init__.py after that first qmerge)
Created attachment 156177 [details, diff] let digestcheck() succeed with empty Manifest and strict mode disabled This should handle an empty Manifest. Are there any other cases left to cover?
an empty Manifest really wasnt what i was after ... a Manifest with missing or different entries is what is useful in other words, what i often do during a version bump is: ebuild <new ebuild> fetch clean unpack here there will be no entries for the new files
Hmm, I think maybe we have a conflict with the expectations of bug 214879. I've always thought that it made sense to make -strict disable all checks but apparently all do not agree on this. Perhaps we should use some flag to toggle the behavior on and off?
i dont think portage can, but i'd expect the behavior to be different with `emerge` and `ebuild`
We can make emerge/ebuild behavior different if we want, but bug 214879 was about ebuild behavior as well.
if you want to address the issue by adding a new FEATURE, that's fine (ebuild-skipmanifest). there has to be a way for the digest checking to be bypassed as being forced to constantly keep them in sync with the files you're developing is a huge pita. maybe i'm a special case, but constantly recomputing digests on large tarballs (kernel/gcc/glibc/etc...) is a killer.
Maybe it shouldn't be a global feature because it's only applies to the ebuild command. How about a --skip-manifest option for the ebuild command? I suppose we could create an EBUILD_DEFAULT_OPTS variable for storing options like that if you want them to be persistent.
a default opts variable would be a must for that path, but otherwise that sounds like a good idea to me
This is fixed in 2.2_rc1.
thanks, seems to work like a charm ... even skips the painful checks on ${A} :D
