Ticket #4270 (closed task: fixed)
Prepare for release mc-4.8.28
Reported by: | zaytsev | Owned by: | zaytsev |
---|---|---|---|
Priority: | major | Milestone: | 4.8.28 |
Component: | adm | Version: | master |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Branch state: | merged | Votes for changeset: |
Description
Attachments
Change History
comment:2 Changed 3 years ago by andrew_b
- Branch state changed from no branch to merged
Merged to master: [eff779b246e667d3bdccb3a0ac2c6dca470ac624].
git lof ---pretty=oneline b8262f22e..eff779b24
comment:3 Changed 3 years ago by andrew_b
Branch: 4270_cleanup
Initial changeset:1a093a8125e109807c87280a4ef9ef458020a6a4
comment:4 Changed 3 years ago by andrew_b
mc-4270-remove-unused-macros.patch: applied.
mc-4270-replace-obsolete-ac_prog_cc_stdc.patch: applied.
Thanks!
comment:6 Changed 3 years ago by and
I bump 4270_cleanup at scan.coverity.com to see was new in Coverity 2021.12.1 release.
Looks like coverity view defects login is still broken.
comment:9 Changed 3 years ago by andrew_b
Merged to master: [7fe3f4c898d029e87faa439ebd7297214059f710].
git --log --pretty=oneline cbcda8a28..7fe3f4c89
comment:10 Changed 3 years ago by andrew_b
- Blocking 4330 removed
(In #4330) Fixed as [185c80ffb79ca3506c6a38a341cb724642b4120c].
comment:11 Changed 3 years ago by zaytsev
Hi Andrew,
I get the following warning while building:
../../../src/viewer/coord_cache.c: In function ‘mcview_ccache_add_entry’: ../../../src/viewer/coord_cache.c:97:5: warning: ‘g_memdup’ is deprecated: Use 'g_memdup2' instead [-Wdeprecated-declarations] 97 | cache->cache[pos] = g_memdup (entry, sizeof (*entry)); | ^~~~~ In file included from /usr/include/glib-2.0/glib.h:82, from ../../../lib/global.h:66, from ../../../src/viewer/coord_cache.c:57: /usr/include/glib-2.0/glib/gstrfuncs.h:257:23: note: declared here 257 | gpointer g_memdup (gconstpointer mem, | ^~~~~~~~
Yury
comment:12 Changed 3 years ago by andrew_b
g_memdup has been deprecated since version 2.68 and should not be used in newly-written code. Use g_memdup2() instead, as it accepts a gsize argument for byte_size, avoiding the possibility of overflow in a gsize → guint conversion
It's not critical in this case, because sizeof (*entry) is significantly less than SIZE_MAX.
Let's fix that after release.
comment:13 Changed 3 years ago by zaytsev
Fine with me - another one from ossie:
> find.c: In function ‘find_cmd’: > find.c:1837:28: warning: ‘start_dir_len’ may be used uninitialized in this function [-Wmaybe-uninitialized] > 1837 | p = name + (size_t) start_dir_len; > | ^~~~~~~~~~~~~~~~~~~~~~ > find.c:1897:13: note: ‘start_dir_len’ was declared here > 1897 | ssize_t start_dir_len; > | ^~~~~~~~~~~~~
For the record.
comment:14 Changed 3 years ago by zaytsev
- Owner set to zaytsev
- Status changed from new to accepted
prepare repository for release
download PO-translations from Transifex.net
store translations in git repo
download the hint translations from Transifex.net
store translations in git repo
create new NEWS wiki page for next version with empty template
add content of current NEWS wiki page to the doc/NEWS file in git repo
new version in Trac
new milestone in Trac
create new tag in git
create tar.(bz2|xz) package files
done
upload source packages and checksums to the special upload area
developers should download tarballs, verify checksums, compile and install locally
skipped
upload source packages and checksums
run command
update Wiki start page with latest release number
done
comment:15 Changed 3 years ago by zaytsev
- Status changed from accepted to testing
- Resolution set to fixed
write an announcement: list user visible changes (bugs and features)
done
create new ticket (type=task, component=adm) for the next release
close ticket for release
close current milestone
done
Branch: 4270_cleanup
Initial changeset:7341eeb52386ec1949bb184018adfaf7c9041fd3