Opened 16 years ago

Closed 16 years ago

Last modified 15 years ago

#9488 closed Patch - Bug Fix (fixed)

convert version.sh for git versions that don't support the --dirty option

Reported by: yiannividalis@… Owned by: beirdo
Priority: minor Milestone: 0.24.1
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Hi,

I wrote in the lists ​http://www.gossamer-threads.com/lists/mythtv/users/468453 about my git version not supporting the 'dirty' option.

I patched version.sh so that it returns the 'git describe' without the 'dirty' option instead of just Unknown.

If it is acceptable, I am offering the patch

Yianni

Attachments (1)

vers.diff​ (358 bytes ) - added by yianniv 16 years ago.

Download all attachments as: .zip

Change History (6)

by yianniv, 16 years ago

Attachment: vers.diff​ added

comment:1 by beirdo, 16 years ago

This is a very poor fix for the problem at hand. You should be using a newer version of git. We don't support anything older than git 1.7.x. I will however put a modified version of this in place.

comment:2 by beirdo, 16 years ago

Milestone: unknown → 0.24.1
Owner: set to beirdo
Status: new → assigned

Modified version committed in 7af8c9a5bc6d (on master) and ffb13127ad14 (on fixes/0.24)

comment:3 by beirdo, 16 years ago

Resolution: → Fixed
Status: assigned → closed

comment:4 by beirdo, 16 years ago

Make that 3f991fa41a030c on fixes/0.24, my bad.

comment:5 by Gavin Hurlbut, 15 years ago

Resolution: Fixed → fixed

Allow braindead old git versions to give version

Since some people insist on using old versions of git (earlier than 1.7.x) that don't support git describe --dirty, I'm putting in a fallback.

Thanks to yiannividalis@… for the idea

Closes #9488

Changeset: 7af8c9a5bc6dff87c02783c34d872e21ee8b0101

Note: See TracTickets for help on using tickets.