Ticket #403 (closed enhancement: fixed) — at Version 10
version.sh fixes
Reported by: | ossi | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 4.7 |
Component: | mc-core | Version: | master |
Keywords: | committed-master | Cc: | |
Blocked By: | Blocking: | #411 | |
Branch state: | Votes for changeset: |
Description (last modified by slyfox) (diff)
Tip: changeset:22ca1a9263d77303cc78518d40e55bd3571918ad
Start: changeset:9a5c9cbc8898a80a82705aacc5be9c5866722b35
git log 22ca1a926^..22ca1a926
Change History
Changed 15 years ago by ossi
- Attachment 0001-remove-useless-commands-which-also-happen-to-be-bash.patch added
comment:1 follow-ups: ↓ 3 ↓ 4 Changed 15 years ago by slyfox
0001-remove-useless-commands-which-also-happen-to-be-bash.patch
All patches look good except the first. It seems to break out-of-tree build. version.h must be updated in source tree even if configure is ran in another directory.
--git-dir might be needed
Mind to try 'git send-email' to mc-bugs@ as a followup to #403 ?
(Message-ID: <042.8aa6aaeb3f2833074e5b63f4e6b7dab8@…>)
comment:2 Changed 15 years ago by slyfox
--git-dir might be needed
or plain old_pwd="$PWD"; cd "${top_src_dir}"; ... cd "$old_pwd"
comment:3 in reply to: ↑ 1 Changed 15 years ago by iNode
About first patch:
pushd and popd does not work in zsh, maybe just remember pwd and return when you need?
comment:4 in reply to: ↑ 1 Changed 15 years ago by ossi
Replying to slyfox:
It seems to break out-of-tree build. version.h must be updated in source tree even if configure is ran in another directory.
whoops, the directory name "maint" triggered an "irrelevant for usual builds" switch in me.
--git-dir might be needed
indeed. patches following ...
Mind to try 'git send-email' to mc-bugs@ as a followup to #403 ?
i miss the point of the exercise ...
Changed 15 years ago by ossi
- Attachment 0001-don-t-suppress-error-messages-from-git-availability-.patch added
replace previous 0002
comment:5 Changed 15 years ago by slyfox
- Keywords review added
Pulled all patches and fixed actual version.h regeneration.
Added forced version.h generation in autogen.sh to avoid double
./configure start in clean envoronment (related to #411)
Branch: 403_versioning_cleanup
Start changeset:9a5c9cbc8898a80a82705aacc5be9c5866722b35
Tip changeset:eea108377d76d7855c25270c2551019098f46070
{{
git log 3337c39..eea10837
}}
comment:10 Changed 15 years ago by slyfox
- Status changed from new to closed
- Keywords committed-master added; vote-iNode vote-slavazanko approved removed
- Resolution set to fixed
- Description modified (diff)