Opened 18 years ago

Closed 18 years ago

Last modified 6 years ago

#8556 closed patch

Amiga version tag

Reported by: SF/polluks Owned by: fingolfin
Priority: normal Component: Port: AmigaOS4
Version: Keywords:
Cc: Game:

Description

--- base/version.cpp.bak Wed Jun 21 21:17:37 2006 +++ base/version.cpp Fri Jul 14 01:01:05 2006 @@ -57,8 +57,8 @@ */ const char *gScummVMVersion = SCUMMVM_VERSION; const char *gScummVMBuildDate = __DATE__ " " __TIME__; -const char *gScummVMVersionDate = SCUMMVM_VERSION " (" __DATE__ " " __TIME__ ")"; -const char *gScummVMFullVersion = "ScummVM " SCUMMVM_VERSION " (" __DATE__ " " __TIME__ ")"; +const char *gScummVMFullVersion = "$VER: ScummVM " SCUMMVM_VERSION " (" __DATE__ " " __TIME__ ")" + sizeof "$VER:"; +const char *gScummVMVersionDate = gScummVMFullVersion + sizeof "ScummVM"; const char *gScummVMFeatures = "" #ifdef USE_TREMOR "Tremor "

Ticket imported from: #1524702. Ticket imported from: patches/661.

Attachments (1)

version.cpp.patch (680 bytes ) - added by SF/polluks 18 years ago.
transparent Amiga version tag

Download all attachments as: .zip

Change History (16)

comment:1 by fingolfin, 18 years ago

Ah come on, you should be able to at least write a single sentence explaining the purpose of your patch. And attach the patch as a file, so that it can be applied / reviewed properly.

Submitting patches w/o any comment or explanation is a waste of time for the patch reviewer, and such patches end up in the garbage bin more often than in a source repository (turning them into a waste of time for the patch submitter, too). I thought I made that clear in our previous mail exchange <sigh>.

comment:2 by fingolfin, 18 years ago

Status: newpending

comment:3 by SF/sf-robot, 18 years ago

This Tracker item was closed automatically by the system. It was previously set to a Pending status, and the original submitter did not respond within 14 days (the time period specified by the administrator of this Tracker).

comment:4 by SF/sf-robot, 18 years ago

Status: pendingclosed

comment:5 by SF/polluks, 18 years ago

This patch adds a transparent version tag for AmigaOS, MorphOS and AROS, something like CVS ident command. You can get the version without execution.

comment:6 by SF/polluks, 18 years ago

Status: closednew

comment:7 by fingolfin, 18 years ago

Still waiting for a *patch*, made against current SVN -- the version.cpp you attached is outdated.

Furthermore, it is not acceptable to modify gScummVMFullVersion like that, w/o having any comment explaining that odd code.

comment:8 by fingolfin, 18 years ago

Owner: set to fingolfin
Status: newpending

by SF/polluks, 18 years ago

Attachment: version.cpp.patch added

transparent Amiga version tag

comment:9 by SF/polluks, 18 years ago

This patch doesn't make anything fancy: > version scummvm-original Could not find version information for 'scummvm-original' > version scummvm-patched ScummVM 0.9

The pointer of gScummVMFullVersion needs an offset to ignore the tag, gScummVMVersionDate just reuses the string, fine?

comment:10 by SF/polluks, 18 years ago

Status: pendingnew

comment:11 by fingolfin, 18 years ago

Status: newpending

comment:12 by fingolfin, 18 years ago

That patch applies against HEAD SVN, very good!

However, it still misses an explanatory comment. I.e. anybody looking at that code will be confused by the "$VER:" prefix. It's mandatory that a comment is added that explains *why* this prefix is added, else it'll be removed again during the next code audit.

comment:13 by SF/sf-robot, 18 years ago

This Tracker item was closed automatically by the system. It was previously set to a Pending status, and the original submitter did not respond within 14 days (the time period specified by the administrator of this Tracker).

comment:14 by SF/sf-robot, 18 years ago

Status: pendingclosed

comment:15 by digitall, 6 years ago

Component: Port: AmigaOS4
Note: See TracTickets for help on using tickets.