From: Ivan Zakharyaschev <imz@altlinux.org> To: darktemplar@altlinux.org, ldv@altlinux.org, devel@lists.altlinux.org Subject: [devel] sisyphus APT considered one more time Date: Fri, 14 Feb 2020 21:59:09 +0300 (MSK) Message-ID: <alpine.LFD.2.20.2002142116020.6363@imap.altlinux.org> (raw) [-- Attachment #1: Type: text/plain, Size: 4378 bytes --] 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
reply other threads:[~2020-02-14 18:59 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=alpine.LFD.2.20.2002142116020.6363@imap.altlinux.org \ --to=imz@altlinux.org \ --cc=darktemplar@altlinux.org \ --cc=devel@lists.altlinux.org \ --cc=ldv@altlinux.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git