This is the mail archive of the ecos-bugs@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug 1001569] Cortex-M arch. whitespace cleanup.


Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001569

--- Comment #3 from Ilija Kocho <ilijak@siva.com.mk> 2012-04-23 17:51:50 BST ---
(In reply to comment #2)
> I don't disagree with the principle and I have been looking forward to doing
> this sort of thing myself, but would you mind this being held back until we
> move to mercurial? Having these sweeping changes across many files is much
> easier to deal with, and less noisy, once we have a system which groups such
> changes into a single changeset.

This clean-up has a purpose and it is to make provision for clean patch(es)
that I intend to post soon. I don't know the prospective for switching to
mercurial but I'm afraid it won't be in time-frame that I need (surprise me :)

The alternatives for this patch (unless there is a pleasent surprise) are
either to merge the clean-up with contribution or post "dirty" patches which
IMO both will produce more noise than this patch.
I am aware that CVS keeps different version number for each file but the
proposed patch is limited to a single package (with single ChangeLog).

> 
> There was a conversation before about fixing the trailing whitespace in
> copyright banners, and I suggested waiting before doing that for the same
> reason. At that point, we could probably remove all trailing whitespace from
> the entire repo (excepting any binary files), not just cortex-m.

Having that in mind I have left them intact.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]