ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-commits: [Wireshark-commits] master-1.8 d70016f: Strip out carriage returns when reading

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 3 Mar 2014 21:55:19 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d70016fdd64f05093863c7fe66fe4c5f7343d8c0
Submitter: Gerald Combs (gerald@xxxxxxxxxxxxx)
Changed: branch: master-1.8
Repository: wireshark

Commits:

d70016f by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Strip out carriage returns when reading version.conf.
    
    The input record separator ($/) under Cygwin's Perl appears to be "\n"
    which means that chomp() won't strip out carriage returns. Make sure
    we do that when reading version.conf, otherwise we can end up with
    unexpected behavior.
    
    Check pkg_enable using an integer comparison as an extra level of
    protection.
    
    (We need to handle CRs better in general but for now we need to get
    version stamping working for the next release.)
    
    Change-Id: I19974467d1a7386728b69dca77f74e678d66ca6b
    Reviewed-on: https://code.wireshark.org/review/483
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    

Actions performed:

    from  aff99aa   More git revision updates.
    adds  d70016f   Strip out carriage returns when reading version.conf.


Summary of changes:
 make-version.pl |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)