Merging GRFCodec & NFORenum sources
Moderator: Graphics Moderators
Merging GRFCodec & NFORenum sources
The GRFCodec and NFORenum sources are sharing some data; actually they have a file in common for some action 2, 7 and D escapes (e.g. \7!) which should be synced between GRFCodec and NFORenum. To ease this we intend to merge the source packages. For build requirements it doesn't matter anything as both GRFCodec and NFORenum need BOOST.
As a result of this the GRFCodec and NFORenum packages (releases and nightlies) will be merged as well which should not be a problem. Yes you might download a package that's slightly bigger but you'll have both GRFCodec and NFORenum updated.
The "problem" we have is the naming of the new package. We're not sure what way to go; keep the GRFCodec name for the whole package, even when it includes NFORenum, or use another slightly more descriptive name such as grfdevtools? However grfdevtools might imply that it includes other grf development tools such as grf2html and grfmaker as well. Does anyone have an opinion about which name to use name or maybe a better name?
As a result of this the GRFCodec and NFORenum packages (releases and nightlies) will be merged as well which should not be a problem. Yes you might download a package that's slightly bigger but you'll have both GRFCodec and NFORenum updated.
The "problem" we have is the naming of the new package. We're not sure what way to go; keep the GRFCodec name for the whole package, even when it includes NFORenum, or use another slightly more descriptive name such as grfdevtools? However grfdevtools might imply that it includes other grf development tools such as grf2html and grfmaker as well. Does anyone have an opinion about which name to use name or maybe a better name?
Re: Merging GRFCodec & NFORenum sources
I would keep GRFCodec.
Formerly known as r0b0t_b0y2003, robotboy, roboboy and beclawat. The best place to get the most recent nightly builds of TTDPatch is: http://roboboy.users.tt-forums.net/TTDPatch/nightlies/
Re: Merging GRFCodec & NFORenum sources
+1robotboy wrote:I would keep GRFCodec.
oh, and a poll might be useful as well (stops people like me making mostly useless posts )
AroAI - A really feeble attempt at an AI
It is practically impossible to teach good programming to students that have had a prior exposure to BASIC: as potential programmers they are mentally mutilated beyond hope of regeneration. --Edsger Dijkstra
It is practically impossible to teach good programming to students that have had a prior exposure to BASIC: as potential programmers they are mentally mutilated beyond hope of regeneration. --Edsger Dijkstra
-
- Tycoon
- Posts: 5948
- Joined: 27 Apr 2005 07:09
- Contact:
Re: Merging GRFCodec & NFORenum sources
As I understand it, this could be "only" a problem with the source which has to be released due to both programs being GPLed. Except from this, I don´t understand why "syncing" two programs but distributing them individually should be a problem at all.Rubidium wrote: The GRFCodec and NFORenum sources are sharing some data; actually they have a file in common for some action 2, 7 and D escapes (e.g. \7!) which should be synced between GRFCodec and NFORenum. To ease this we intend to merge the source packages. [...]
Anyway, in case this should happen, I´d say stay with grfcodec, simply because this is the main tool, nforenum being "only" an auxiliary tool, i.e. not mandatory.
regards
Michael
Re: Merging GRFCodec & NFORenum sources
GRFCodec++
wallyweb on tt-forums: Screenshots - Wallyweb World - Projects & Releases
wallyweb on Simuscape: Projects - Releases
Other Stuff: TTDPatch 2.6 "Nightly" download - cirdan's OpenTTD branch (New Map Features)
Screenshot Of The Month Contest Winner: August 2015 - Tied May 2016 - January 2018 - December 2018 - May 2019
wallyweb on Simuscape: Projects - Releases
Other Stuff: TTDPatch 2.6 "Nightly" download - cirdan's OpenTTD branch (New Map Features)
Screenshot Of The Month Contest Winner: August 2015 - Tied May 2016 - January 2018 - December 2018 - May 2019
- planetmaker
- OpenTTD Developer
- Posts: 9432
- Joined: 07 Nov 2007 22:44
- Location: Sol d
Re: Merging GRFCodec & NFORenum sources
grfcodec
OpenTTD: manual | online content | translations | Wanted contributions and patches
#openttdcoop: blog | wiki | public server | DevZone | NewGRF web translator
DevZone - home of the free NewGRFs: OpenSFX | OpenMSX | OpenGFX | Swedish Rails | OpenGFX+ Trains|RV|Industries|Airports|Landscape | NML
Re: Merging GRFCodec & NFORenum sources
Actually, then anyone (even those that don't use it at all) votes anonymously and I still don't have a clue what the actual users/people with interest in it think.Lord Aro wrote:poll might be useful as well
This has the same problems as grfdevtools (maybe even slightly more).FooBar wrote:GRFTools
It's not a problem, it's more a hassle; the effort for a release is essentially doubled at all places in the chain. Checking changelogs, readmes and such in two repositories, writing two release notes, people having to download two packages, downstreams (distros) having to update two packages.michael blunck wrote:I don´t understand why "syncing" two programs but distributing them individually should be a problem at all.
It would be perfectly possible to create two binary packages though, but in my opinion the drawbacks of that outweigh the benefits. Downstreams are still able to keep two different packages if they like to.
That's not significantly better/more descriptive than GRFCodec, though the best alternative to GRFCodec I've seen so far. In any case I was intending to bump the version significantly if we'd stay with GRFCodec; something 5.x.y, so it's bigger than NFORenum's and GRFCodec's version.wallyweb wrote:GRFCodec++
Re: Merging GRFCodec & NFORenum sources
I like that as well if not more than GRFCodec.wallyweb wrote:GRFCodec++
Formerly known as r0b0t_b0y2003, robotboy, roboboy and beclawat. The best place to get the most recent nightly builds of TTDPatch is: http://roboboy.users.tt-forums.net/TTDPatch/nightlies/
Re: Merging GRFCodec & NFORenum sources
How about dropping the "grf" part?
There are various ways to create grfs, with nml or with grfmaker.
Otoh nforenum does not deal with grfs at all.
So I would rather go for "nfotools", "nfokit", ...
There are various ways to create grfs, with nml or with grfmaker.
Otoh nforenum does not deal with grfs at all.
So I would rather go for "nfotools", "nfokit", ...
⢇⡸⢸⠢⡇⡇⢎⡁⢎⡱⢸⡱⢸⣭⠀⢸⢜⢸⢸⣀⢸⣀⢸⣭⢸⡱⠀⢰⠭⡆⣫⠰⣉⢸⢸⠀⢰⠭⡆⡯⡆⢹⠁⠀⢐⠰⡁
Re: Merging GRFCodec & NFORenum sources
I kinda like the GRF part myself. Am I right in assuming Codec stands for compiler and decompiler?
GRFCode, GRFCode+ - I like shorter names.
GRFCom - GRF Compiler
GRFCom+ - GRF Compiler and more
GRFPro - GRF Processor or GRF Professional
GRFCode, GRFCode+ - I like shorter names.
GRFCom - GRF Compiler
GRFCom+ - GRF Compiler and more
GRFPro - GRF Processor or GRF Professional
-
- Tycoon
- Posts: 5948
- Joined: 27 Apr 2005 07:09
- Contact:
Re: Merging GRFCodec & NFORenum sources
Well, if it´s only about the name of the package, anything could work, given that both programs retain their original name. BTW, "grfcodec" is an already well-known term in the Unix/Linux world, have a look at google.Rubidium wrote: It's not a problem, it's more a hassle [...]
regards
Michael
Re: Merging GRFCodec & NFORenum sources
I would consider the merge as grfcodec getting extended functionality, namely annotated nfo output. You should probably add flags to allow the user to specify what he wants.
To ease the mind of people wanting to keep both tools, you could check argv[0] to decide on the default flag option settings, a trick often employed to deploy several closely related programs:For people less familiar with the -i option of ls, the first number in both lines is the inode number, the unique ID number of a file at a file system. Since it is the same, both programs are really the same file, it just has two names. Depending on which name you use to start the program, it does something different.
To ease the mind of people wanting to keep both tools, you could check argv[0] to decide on the default flag option settings, a trick often employed to deploy several closely related programs:
Code: Select all
$ ls -li /bin/ed /bin/red
87585 -rwxr-xr-x. 2 root root 53352 Jul 25 2009 /bin/ed
87585 -rwxr-xr-x. 2 root root 53352 Jul 25 2009 /bin/red
- planetmaker
- OpenTTD Developer
- Posts: 9432
- Joined: 07 Nov 2007 22:44
- Location: Sol d
Re: Merging GRFCodec & NFORenum sources
In the light that these tools basically deal with the nfo language (and considering that most tools tell what they take as input, e.g. asm (assember files) ,gcc, msvc (c files), photoshop (photos), ... and not their output (binary, exe, graphics), frosch's proposal to call it like 'nfotools', 'nfokit' or maybe 'nfoc' might seem the more logical choice.
Alberth's proposal might seem a good idea, too; though I'm not sure how far that can be realized on windows systems.
Alberth's proposal might seem a good idea, too; though I'm not sure how far that can be realized on windows systems.
OpenTTD: manual | online content | translations | Wanted contributions and patches
#openttdcoop: blog | wiki | public server | DevZone | NewGRF web translator
DevZone - home of the free NewGRFs: OpenSFX | OpenMSX | OpenGFX | Swedish Rails | OpenGFX+ Trains|RV|Industries|Airports|Landscape | NML
Re: Merging GRFCodec & NFORenum sources
I'm not talking about merging the GRFCodec and NFORenum binaries, just their source packages. There will still be a grfcodec and nforenum (and grfmerge/grfdiff) binaries after the merger.
NFOTools might be an alternative name, although it doesn't quite cover grfmerge and grfdiff. I'd still favour GRFCodec.
NFOTools might be an alternative name, although it doesn't quite cover grfmerge and grfdiff. I'd still favour GRFCodec.
- andythenorth
- Tycoon
- Posts: 5658
- Joined: 31 Mar 2007 14:23
- Location: Lost in Music
Re: Merging GRFCodec & NFORenum sources
GRFCodec +1
FIRS Industry Replacement Set (released) | HEQS Heavy Equipment Set (trucks, industrial trams and more) (finished)
Unsinkable Sam (ships) (preview released) | CHIPS Has Improved Players' Stations (finished)
Iron Horse ((trains) (released) | Termite (tracks for Iron Horse) (released) | Busy Bee (game script) (released)
Road Hog (road vehicles and trams) (released)
Unsinkable Sam (ships) (preview released) | CHIPS Has Improved Players' Stations (finished)
Iron Horse ((trains) (released) | Termite (tracks for Iron Horse) (released) | Busy Bee (game script) (released)
Road Hog (road vehicles and trams) (released)
Re: Merging GRFCodec & NFORenum sources
andythenorth wrote:GRFCodec +1
Re: Merging GRFCodec & NFORenum sources
Sorry for the jocularity, but I simply can't resist this ... How about "That Thingamajig Used To Check NFO Code AND Spit Out A GRF" ?Rubidium wrote:I'm not talking about merging the GRFCodec and NFORenum binaries, just their source packages. There will still be a grfcodec and nforenum (and grfmerge/grfdiff) binaries after the merger.
The point here is that we would be looking at a package that performs two or more functions. The output of this tool is a grf file, and failing that, a suitably commented nfo file that points out the error(s) of our ways, as well as being a tool that can decode a grf file. Consequently, something in keeping with GRFCodec would seem most appropriate. "GRFWizard" has already been claimed.
wallyweb on tt-forums: Screenshots - Wallyweb World - Projects & Releases
wallyweb on Simuscape: Projects - Releases
Other Stuff: TTDPatch 2.6 "Nightly" download - cirdan's OpenTTD branch (New Map Features)
Screenshot Of The Month Contest Winner: August 2015 - Tied May 2016 - January 2018 - December 2018 - May 2019
wallyweb on Simuscape: Projects - Releases
Other Stuff: TTDPatch 2.6 "Nightly" download - cirdan's OpenTTD branch (New Map Features)
Screenshot Of The Month Contest Winner: August 2015 - Tied May 2016 - January 2018 - December 2018 - May 2019
Who is online
Users browsing this forum: No registered users and 21 guests