Bug 116807 - Update port: textproc/sxml
Summary: Update port: textproc/sxml
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Stefan Walter
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-02 07:50 UTC by Kouichi ABE
Modified: 2007-10-06 12:30 UTC (History)
1 user (show)

See Also:


Attachments
sxml-ports.tar.gz (1.25 KB, text/plain)
2007-10-02 07:50 UTC, Kouichi ABE
no flags Details
sxml.diff (1.90 KB, patch)
2007-10-05 18:25 UTC, Kouichi ABE
no flags Details | Diff
sxml.diff (1.70 KB, patch)
2007-10-06 12:07 UTC, Kouichi ABE
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kouichi ABE 2007-10-02 07:50:07 UTC
	I update sxml version from 1.0.4 to 1.0.5.
	The wyvern-2.2.10 in www/wyvern depends on sxml-1.0.5.
Comment 1 Stefan Walter freebsd_committer freebsd_triage 2007-10-05 18:07:11 UTC
Hi Kouichi,

could you resend this one as a unified diff, too?

Thanks,
Stefan
Comment 2 Stefan Walter freebsd_committer freebsd_triage 2007-10-05 18:07:25 UTC
State Changed
From-To: open->feedback

Awaiting submitter's feedback. 


Comment 3 Stefan Walter freebsd_committer freebsd_triage 2007-10-05 18:07:25 UTC
Responsible Changed
From-To: freebsd-ports-bugs->stefan

Take.
Comment 4 Kouichi ABE 2007-10-05 18:25:46 UTC
Hi Stefan,

Sorry, I forgot to resend the patch.
I resend it in unified diff format.

Best regards.
-- 
Kouichi ABE (WALL)
kouichi@MysticWALL.COM
http://www.MysticWALL.COM/
4196 185C B814 8F96 D170  B492 71A8 1B2B B9D5 5CF5
Comment 5 Stefan Walter freebsd_committer freebsd_triage 2007-10-06 09:44:14 UTC
Hi Kouichi,

there are still a few issues with this update:

- You removed USE_LDCONFIG, but the port installs shared libraries. When
  installing shared libraries, USE_LDCONFIG is usually desired.
- You're setting PREFIX to LOCALBASE. This is extremely discouraged. The
  port should install to the PREFIX defined by the user/system.
- The patch files/patch-sxmlrpc.c fails to apply with the new version.
- I'm not sure if NO_MTREE is really needed here.

Could you look at these and send me another update?

Thanks and best regards,
Stefan
Comment 6 Kouichi ABE 2007-10-06 12:07:18 UTC
Hi Stefan,

On Sat, Oct 06, 2007 at 10:47:36AM +0000, Stefan Walter wrote
> 
> - You removed USE_LDCONFIG, but the port installs shared libraries. When
>   installing shared libraries, USE_LDCONFIG is usually desired.
>
> - You're setting PREFIX to LOCALBASE. This is extremely discouraged. The
>   port should install to the PREFIX defined by the user/system.
>
> - I'm not sure if NO_MTREE is really needed here.

Sorry, I made a mistake to modify the sxml-port.


> - The patch files/patch-sxmlrpc.c fails to apply with the new version.

The patch is alredy included in the sxml-1.0.5.
I forgot to use the option '-N' for the diff command.

I resend new patch in unified diff format.

Best regards.
-- 
Kouichi ABE (WALL)
kouichi@MysticWALL.COM
http://www.MysticWALL.COM/
4196 185C B814 8F96 D170  B492 71A8 1B2B B9D5 5CF5
Comment 7 dfilter service freebsd_committer freebsd_triage 2007-10-06 12:23:58 UTC
stefan      2007-10-06 11:23:52 UTC

  FreeBSD ports repository

  Modified files:
    textproc/sxml        Makefile distinfo 
  Removed files:
    textproc/sxml/files  patch-sxmlrpc.c 
  Log:
  Update to 1.0.5.
  
  PR:             116807
  Submitted by:   maintainer
  
  Revision  Changes    Path
  1.2       +1 -1      ports/textproc/sxml/Makefile
  1.2       +3 -3      ports/textproc/sxml/distinfo
  1.2       +0 -23     ports/textproc/sxml/files/patch-sxmlrpc.c (dead)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 8 Stefan Walter freebsd_committer freebsd_triage 2007-10-06 12:24:03 UTC
State Changed
From-To: feedback->closed

Committed, thanks!