Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#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)

patch (307 bytes) - added by IgnitusBoyone@… 8 years ago.
Port Patch

Download all attachments as: .zip

Change History (6)

comment:1 Changed 8 years ago by jmroot (Joshua Root)

Owner: changed from macports-tickets@… to mccdo@…
Port: gmtl added
Version: 1.9.2

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.

Changed 8 years ago by IgnitusBoyone@…

Attachment: patch added

Port Patch

comment:2 Changed 8 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 8 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 8 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed

comment:5 Changed 8 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.

Note: See TracTickets for help on using tickets.