Ticket #145 (closed enhancement: fixed)
remove stuff created by autogen.sh
| Reported by: | Enrico Weigelt <weigelt@…> | Owned by: | slavazanko |
|---|---|---|---|
| Priority: | major | Milestone: | 4.7 |
| Component: | mc-core | Version: | 4.6.2 |
| Keywords: | commited-master | Cc: | |
| Blocked By: | Blocking: | ||
| Branch state: | Votes for changeset: |
Description (last modified by metux) (diff)
The files in m4/* seem to be a bit outdated, at least they get
changed by the autogen process.
Attachments
Change History
comment:2 Changed 16 years ago by slavazanko
- Description modified (diff)
Is we need to ignore these files (via .gitignore)?
comment:3 Changed 16 years ago by anonymous
this patch updates the m4/*.m4 files to whats changed by autogen.sh
comment:4 Changed 16 years ago by metux
- Keywords review added
- Type changed from defect to enhancement
- Description modified (diff)
- Milestone set to 4.6.2
comment:6 Changed 16 years ago by winnie
- Keywords vote-winnie added
Exact the same changes I see on my box also. .this is very nasty.. therefore +1 from me.
comment:7 Changed 16 years ago by ossi
actually, it is generally considered bad practice to have generated files under version control at all. anything created by the bootstrap process (autogen.sh) should be purged from the tree.
comment:8 Changed 16 years ago by winnie
- Keywords rework added; review vote-winnie removed
- Summary changed from update m4/* files to remove stuff created by autogen.sh
well this sounds right. I'm retitling this bug and added the rework keyword. metux do you want to update the patch or should I?
Changed 16 years ago by slavazanko
- Attachment 145_update_m4_files-rev2.patch added
Remove autogenerated m4/*.m4 files. Some other changes for autocreate *.m4 files
comment:9 Changed 16 years ago by slavazanko
- Keywords rewiew added; rework removed
see 145_update_m4_files-rev2.patch
Need to review.
comment:10 Changed 16 years ago by slavazanko
- Milestone changed from 4.6.2 to 4.7
This ticket not a bugreport- this an enhancement.
Therefore need to set milestone 4.7
comment:11 Changed 16 years ago by slavazanko
please, see 145_update_m4_files branch (based on master)
comment:14 Changed 16 years ago by styx
- Owner changed from metux to slavazanko
- Status changed from accepted to assigned
comment:17 Changed 16 years ago by slyfox
- Keywords voty-slyfox approved added; review removed
Looks really good and usable!
comment:18 Changed 16 years ago by slavazanko
- Status changed from accepted to testing
- Keywords commited-master added; vote-styx voty-slyfox approved removed
- Resolution set to fixed
Fixed.
This message has 0 attachment(s)