Bug 192137 - [MAINTAINER UPDATE] lang/tcc
Summary: [MAINTAINER UPDATE] lang/tcc
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: i386 Any
: --- Affects Only Me
Assignee: Guido Falsi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-07-26 16:01 UTC by Carlos J Puga Medina
Modified: 2014-07-29 18:52 UTC (History)
2 users (show)

See Also:


Attachments
tcc.diff (1.67 KB, patch)
2014-07-26 16:01 UTC, Carlos J Puga Medina
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos J Puga Medina 2014-07-26 16:01:13 UTC
Created attachment 145000 [details]
tcc.diff

- Clean-up Makefile
- Add INFO macro
- Fix pkg-plist
- Bump PORTREVISION
Comment 1 Carlos J Puga Medina 2014-07-26 16:36:24 UTC
Build log:

https://redports.org/buildarchive/20140726161932-16432/

Despite the error on 11-CURRENT/i386. I tested it on my machine and it installs fine.

http://pastebin.com/bXfcAMfP # 11-CURRENT/i386
Comment 2 John Marino freebsd_committer freebsd_triage 2014-07-26 22:43:41 UTC
I think you should state if "tested it on my machine" if "my machine" equals "FreeBSD 11 i386" and if "tested it" means installed locally or properly using poudriere...

Setting to "patch-ready" but you (Carlos) should answer those questions for the person that picks up the PR.
Comment 3 Carlos J Puga Medina 2014-07-26 22:55:56 UTC
(In reply to John Marino from comment #2)
> I think you should state if "tested it on my machine" if "my machine" equals
> "FreeBSD 11 i386" and if "tested it" means installed locally or properly
> using poudriere...
> 
> Setting to "patch-ready" but you (Carlos) should answer those questions for
> the person that picks up the PR.

Yes, I installed it locally. If you need more details, please ask me.
Comment 4 John Marino freebsd_committer freebsd_triage 2014-07-26 23:20:45 UTC
so local installs are not really worth much.  IOW you say you installed it locally, but that doesn't trump a redports failure.

Now if it passes a poudriere testpoint on FreeBSD 11 i386, that would trump a redports failure.
Comment 5 Carlos J Puga Medina 2014-07-26 23:28:29 UTC
Ok, I will prepare the environment to perform a test with poudriere. Currently I don't have it installed neither tinderbox.
Comment 6 Carlos J Puga Medina 2014-07-27 00:00:05 UTC
Meanwhile to advance, please can you test it? I'll return the favor.
Comment 7 John Marino freebsd_committer freebsd_triage 2014-07-27 00:01:53 UTC
I'm just the triage guy, I was only prepping you for the real committer. :)
Comment 8 Carlos J Puga Medina 2014-07-27 00:05:10 UTC
(In reply to John Marino from comment #7)
> I'm just the triage guy, I was only prepping you for the real committer. :)

Thanks mate :)
Comment 9 commit-hook freebsd_committer freebsd_triage 2014-07-29 14:34:19 UTC
A commit references this bug:

Author: madpilot
Date: Tue Jul 29 14:33:18 UTC 2014
New revision: 363329
URL: http://svnweb.freebsd.org/changeset/ports/363329

Log:
  - Fix plist
  - Add INFO macro and USES=makeinfo to make it also work on revent head
  - Clean up Makefile

  PR:		192137
  Submitted by:	Carlos Jacobo Puga Medina <cjpugmed@gmail.com> (maintainer)

Changes:
  head/lang/tcc/Makefile
  head/lang/tcc/pkg-plist
Comment 10 Guido Falsi freebsd_committer freebsd_triage 2014-07-29 14:36:01 UTC
(In reply to commit-hook from comment #9)
> A commit references this bug:
> 
> Author: madpilot
> Date: Tue Jul 29 14:33:18 UTC 2014
> New revision: 363329
> URL: http://svnweb.freebsd.org/changeset/ports/363329
> 
> Log:
>   - Fix plist
>   - Add INFO macro and USES=makeinfo to make it also work on revent head

The USES macro is what fixed your build error on head.

makeinfo has been recently removed from FreeBSD, that macro takes care of checking if it's available or adding a dependency on the relevant port.

I took the freedom to directly commit this one.

Thanks for your submission!
Comment 11 Carlos J Puga Medina 2014-07-29 18:52:55 UTC
(In reply to Guido Falsi from comment #10)
> (In reply to commit-hook from comment #9)
> > A commit references this bug:
> > 
> > Author: madpilot
> > Date: Tue Jul 29 14:33:18 UTC 2014
> > New revision: 363329
> > URL: http://svnweb.freebsd.org/changeset/ports/363329
> > 
> > Log:
> >   - Fix plist
> >   - Add INFO macro and USES=makeinfo to make it also work on revent head
> 
> The USES macro is what fixed your build error on head.
> 
> makeinfo has been recently removed from FreeBSD, that macro takes care of
> checking if it's available or adding a dependency on the relevant port.
> 
> I took the freedom to directly commit this one.
> 
> Thanks for your submission!

Thanks. I didn't realize it, therefore, I forgot to add USE=makeinfo.