#29402 closed update (fixed)
Request GMTL portfile update
Reported by: | IgnitusBoyone@… | Owned by: | mccdo@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | ||
Port: | gmtl |
Description
I found the gmtl port file to be 23 months stale. The original maintainer was from Iowa State and I would assume a graduate student no longer associated with them.
I had to update it to 6.1 for my own system. I was hopping I could push the update upstream.
PortSystem 1.0 name gmtl version 0.6.1 categories math platforms darwin maintainers iastate.edu:mccdo description a lightweight math library long_description The math library used by vrjuggler. It is \ implemented with c++ templates. homepage http://ggt.sourceforge.net/ master_sites sourceforge:ggt checksums md5 1391af2c5ea050dda7735855ea5bb4c1 depends_build port:scons use_configure no build.cmd scons build.target destroot.cmd scons destroot.destdir prefix=${destroot}${prefix} destroot.target install
Attachments (1)
Change History (6)
comment:1 Changed 14 years ago by jmroot (Joshua Root)
Owner: | changed from macports-tickets@… to mccdo@… |
---|---|
Port: | gmtl added |
Version: | 1.9.2 |
comment:2 Changed 14 years ago by IgnitusBoyone@…
I was able to contact iastate.edu:mccdo and he notified me that he was fine with the patch. Do I need to direct him to post directly to the ticket? I attached the patch file and included it inline to this reply. Thanks to your sourceforge pull code its just an md5 change with a version update.
4c4 < version 0.6.1 --- > version 0.5.4 13c13 < checksums md5 1391af2c5ea050dda7735855ea5bb4c1 --- > checksums md5 1ba4ceb8e883254b7d7ebe565c124520 17c17 < use_configure no --- > use_configure no 20c20 < build.target --- > build.target 23a24 >
comment:3 Changed 14 years ago by mccdo@…
That patch works for me. Thanks for the update. Hopefully the committers can get this change in shortly.
comment:4 Changed 14 years ago by jmroot (Joshua Root)
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:5 Changed 14 years ago by jmroot (Joshua Root)
In this case the patch was simple enough that it doesn't really matter, but for future reference, a unified diff means using diff -u
(this is also the format produced by svn diff
). Trac can display that format in a nicer way. Also please try to avoid including whitespace changes and functional changes in the same patch.
Please remember to fill in the Port field. If the maintainer address is no longer valid and a working address for the maintainer can't be found, it should be changed to nomaintainer. OTOH, if it's not certain that the current address doesn't work, it needs to be on the ticket.
It would also make reviewing your changes easier if you supplied them in the form of a unified diff.