From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 14 Feb 2020 21:59:09 +0300 (MSK) From: Ivan Zakharyaschev To: darktemplar@altlinux.org, ldv@altlinux.org, devel@lists.altlinux.org Message-ID: User-Agent: Alpine 2.20 (LFD 67 2015-01-07) MIME-Version: 1.0 Content-Type: multipart/mixed; BOUNDARY="1807885841-1847092450-1581706749=:6363" Subject: [devel] sisyphus APT considered one more time X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 14 Feb 2020 18:59:10 -0000 Archived-At: List-Archive: List-Post: This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --1807885841-1847092450-1581706749=:6363 Content-Type: text/plain; charset=KOI8-R Content-Transfer-Encoding: 8BIT Hello! Могу предложить в сыром виде посмотреть такое представление изменений, соответствующих ветке sisyphus_one_more_time в http://git.altlinux.org/people/darktemplar/packages/apt.git , которое у меня получилось. (У меня в ветке one_more_time в http://git.altlinux.org/people/darktemplar/packages/apt.git .) По смыслу должно то же самое получится в итоге. (Посравниваю git diff -w darktemplar@ALT/sisyphus_one_more_time..imz@ALT/one_more_time ). Кое-что ещё из тех переделок этих изменений, что ещё раньше здесь обсуждалось, я бы внёс сразу, но пока не добрался.) Отделял отдельные изменения для лучшего восприятия, ну и прочее; поправил что-то не очень хорошее. Возникавшие вопросы-замечания уж не буду писать, я их все как-нибудь для себя решил и внёс изменения. Можно увидеть в конкретных коммитах. Менять типы на 32-битный map_ptrloc (упоминавшийся вчера) пока не увидел смысла: в частности не вижу, чтобы как-то чисто обрабатывались переполнения ни в старом коде, ни с map_ptrloc, и не очень хочется одну грязь на другую менять. (Однако, на практике такие размеры графа пакетов не ожидаются, конечно -- чтобы вылезло за 32-битные смещения). (Впрочем, на эту тему ещё заметил, что параметры на размер области из конфигурации могут быть прочитаны только типа int функцией FindI(); правда, область может разрастаться и стать больше.) $ git log --oneline --reverse 0.5.15lorg2-alt69..one_more_time ff29a765b .dir-locals.el: apply our Emacs mode settings to .h files, too 138d903a6 dynamic memory management: Fix off by one error in mmap(...)/munmap(...) leading to resource leak 1695ed045 DynamicMMap::DynamicMMap(): avoid damage to the last byte of the file ba75a61c9 fixes: Fix every use of va_list 85bec48e1 fixes: strutil.cc: rework string iterating c5db927fd fixes: gpg.cc: fix potential memory leak 00f21e001 fixes: RPM ArchiveURI: check file length before using it 4d9aa4d84 fixes: Use clear() function instead of assignment of empty value 4bbb7edae API changes: Use 'override' keyword 8326ce09f API changes: remove function without implementation beda412a9 API changes: Add 'const' modificator to member function a61caa1d5 API changes: Avoid copying objects 84edf4189 dynamic memory management: wrap the mmap actions ae35d9210 dynamic memory management: Use references instead of copies in the Cache generation methods 1095870b0 DynamicMMap::WriteString(): fix the check for "out of room" when length is automatic 767a2bc39 mmap.{h,cc}: better coding style f6c475d73 dynamic memory management: Support large files c830a1658 MMap: it does not make sense to have iSize larger than the system can mmap(2) 17955a25e add MMap::validData() (a short way to write the check) c82ab1152 DynamicMMap::RawAllocate(): do not modify data on failure dcd84fa29 DynamicMMap::RawAllocate(): no offset needed if already aligned (was off-by-one) e451ea0cc use the safer C++-style static_cast instead of a C-style cast (from void*) 9d9b438a3 rigorously compute HeaderP as the pointer to the beginning of Map df7528533 Use special type to return allocation failure since 0 is a valid offset value e11e22987 factor out a repeated translation format 9b5446f2d don't even start to fill in a structure, if we can't allocate the prerequisites 696cbc244 Add workaround for packages with missing tags d976934af DynamicMMap::RawAllocate(): assert that a zero-size item is never allocated 2490cd50d refactor DynamicMMap::WriteString() to re-use DynamicMMap::RawAllocate() 527d18d0b dynamic memory management: apt-pkg/pkgcachegen.{cc,h} changes 3c47cf65e dynamic memory management: Add and document APT::Cache-{Start,Grow,Limit} options for mmap control 5055b1f5c dynamic memory management: DynamicMMap::Grow: add optional debug output 0c34bc3da 0.5.15lorg2-alt70 e04264179 packagekit support: Port pkgCacheFile::GetSourceList and its dependencies from Debian a1621866c packagekit support: Port ListUpdate function from Debian f2e3bdf0d API changes: Remove ABI compat stuff 4fe3d8379 API changes: Bump soname 84ab36cf7 0.5.15lorg2-alt71 c2d779446 rpmindexfile.cc rpmPkgListIndex::Merge(): drop a bogus call to FileFd::Seek() 184fb8470 fixes: Process error on loading release info 927a47b0d fixes: Fix copying release information from cdrom af5c308c4 0.5.15lorg2-alt72 $ git log -p --reverse 0.5.15lorg2-alt69..one_more_time (Тут слишком длинная простыня получилась.) -- Best regards, Ivan --1807885841-1847092450-1581706749=:6363--