Bug 193687 - [PATCH] databases/freetds-msdblib: take maintainership
Summary: [PATCH] databases/freetds-msdblib: take maintainership
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Thomas Zander
URL:
Keywords:
Depends on: 193686
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-16 16:02 UTC by Muhammad Moinur Rahman
Modified: 2014-10-03 06:55 UTC (History)
6 users (show)

See Also:


Attachments
patch for freetds-msdblin-0.91.103 (339 bytes, patch)
2014-09-16 16:02 UTC, Muhammad Moinur Rahman
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Muhammad Moinur Rahman freebsd_committer freebsd_triage 2014-09-16 16:02:48 UTC
Created attachment 147380 [details]
patch for freetds-msdblin-0.91.103

- Take maintainership
Comment 1 Marcus von Appen freebsd_committer freebsd_triage 2014-09-17 16:53:14 UTC
Patch looks good, over to Patch Ready
Comment 2 Thomas Zander freebsd_committer freebsd_triage 2014-09-21 14:07:23 UTC
I would prefer to remove this port after committing the patch in bug 193686:
- this is just one OPTION that can be enabled in freetds, so it looks like an obsolete port after the conversion to options ng
- it can't be installed if freetds is already installed (so we would need CONFLICTS in both port Makefiles anyway)

Objections?
Comment 3 Muhammad Moinur Rahman freebsd_committer freebsd_triage 2014-09-21 14:11:58 UTC
I thought of removing it too. But unfortunately there are dependent ports. 
So CONFLICTS is the way we should follow.

(In reply to Thomas Zander from comment #2)
> I would prefer to remove this port after committing the patch in bug 193686:
> - this is just one OPTION that can be enabled in freetds, so it looks like
> an obsolete port after the conversion to options ng
> - it can't be installed if freetds is already installed (so we would need
> CONFLICTS in both port Makefiles anyway)
> 
> Objections?
Comment 4 Thomas Zander freebsd_committer freebsd_triage 2014-09-21 14:54:17 UTC
(In reply to Muhammad Moinur Rahman from comment #3)
> I thought of removing it too. But unfortunately there are dependent ports. 
> So CONFLICTS is the way we should follow.

Just to make sure I got this right (did some grepping and reading):
1) We have 10 ports that LIB_DEPENDS on libsybdb.so
2) Out of those, some say depend on freetds and some on freetds-msdblib
3) At least one (others as well?) is broken with freetds-msdblib

Regarding 2)
Do these explicitly require MSDBLIB? If so, how could the port ensure build requirements are met? There is only the dependency on libsybdb.so. What if freetds is already installed? The dependency will be satisfied and the build/link/run fail.
Regarding 3)
The sybtcl tarball has not changed in 14 years, so the project may be abandoned upstream and the problem will not go away. If this is the only one that is broken with msdblib, I'd say the best option is to work with the maintainer of sybtcl and get a patch to make this work with MSDBLIB, then remove the OPTION, make it default and retire the freetds-msdblib port.
(It may be worth checking out other package repositories, e.g. from OpenBSD, pkgsrc or Gentoo, a fix may already exist.)
Would you be able to check with the maintainer of sybctl? I added him to the CC list for this bug.
Comment 5 Muhammad Moinur Rahman freebsd_committer freebsd_triage 2014-09-21 15:48:26 UTC
While going through the Makefile it's mentioned that sybctl relies on sybase style dblib. Now if we dig deeper there is a CONFIGURE_ARGS which is 
--enable-sybase-compat. So I will try to incorporate both the ARGS and check whether if sybctl builds. Will let you know at night.

(In reply to Thomas Zander from comment #4)
> (In reply to Muhammad Moinur Rahman from comment #3)
> > I thought of removing it too. But unfortunately there are dependent ports. 
> > So CONFLICTS is the way we should follow.
> 
> Just to make sure I got this right (did some grepping and reading):
> 1) We have 10 ports that LIB_DEPENDS on libsybdb.so
> 2) Out of those, some say depend on freetds and some on freetds-msdblib
> 3) At least one (others as well?) is broken with freetds-msdblib
> 
> Regarding 2)
> Do these explicitly require MSDBLIB? If so, how could the port ensure build
> requirements are met? There is only the dependency on libsybdb.so. What if
> freetds is already installed? The dependency will be satisfied and the
> build/link/run fail.
> Regarding 3)
> The sybtcl tarball has not changed in 14 years, so the project may be
> abandoned upstream and the problem will not go away. If this is the only one
> that is broken with msdblib, I'd say the best option is to work with the
> maintainer of sybtcl and get a patch to make this work with MSDBLIB, then
> remove the OPTION, make it default and retire the freetds-msdblib port.
> (It may be worth checking out other package repositories, e.g. from OpenBSD,
> pkgsrc or Gentoo, a fix may already exist.)
> Would you be able to check with the maintainer of sybctl? I added him to the
> CC list for this bug.
Comment 6 Thomas Zander freebsd_committer freebsd_triage 2014-10-03 06:55:38 UTC
Port has been merged with the freetds port and removed from the tree.