Re: [146276] trunk/dports/databases

2016-03-06 Thread Ryan Schmidt

> On Mar 6, 2016, at 1:22 PM, Mojca Miklavec  wrote:
> 
> On 6 March 2016 at 18:01, Bradley Giesbrecht wrote:
>> 
>> We could remove all ports that depend on mysql4 and mysql5 exclusively or
>> via default variants. The remaining ports would then have their mysql4 or
>> mysql5 variant removed.
>> 
>> The removed ports could be brought back if someone was interested in fixing
>> them.
> 
> That would be a bit harsh, wouldn't it? I bet that a number of ports
> would work with a different database without any additional patches.
> 
> I asked long ago if someone could provide guidelines for transition,
> but nobody did. It's not clear to me what the suggested way is, so
> that we could at least make some consistent changes rather than every
> port switching to a different database with completely different code.
> 
> Mojca

The problem is that Oracle bought MySQL and took it in directions some users 
and developers object to. Therefore, the original developer of MySQL forked it 
and made MariaDB. Some distributions now default to MariaDB, and I would be in 
favor of MacPorts doing that too. But we should use the latest version, which 
is currently 10.1, in the mariadb-10.1 port. Previously we used the port name 
as the variant name, for example +mysql56 to enable mysql56 support. The fact 
that this port name contains a dash and a period is problematic in that dashes 
are not permitted in variant names because that conflicts with the syntax for 
disabling a variant, and the period is not an officially sanctioned character 
to use in a variant name either. Initial tests showed it seemed to work, but 
there are several places in the base code where variant names are validated; 
they all need to be changed to allow a period, if we want to use a period in 
the variant name. Whatever we decide should be applied 
 consistently to all ports, so that they all have the same set of 
MySQL-compatible variants and all default to the same (latest stable) one. Some 
tickets related to these issues:

https://trac.macports.org/ticket/39961

https://trac.macports.org/ticket/43431

https://trac.macports.org/ticket/46807

Bradley, can you add notes to #46807 about what you did so far?

___
macports-dev mailing list
macports-dev@lists.macosforge.org
https://lists.macosforge.org/mailman/listinfo/macports-dev


Re: [146276] trunk/dports/databases

2016-03-06 Thread Bradley Giesbrecht
> On Mar 6, 2016, at 11:22 AM, Mojca Miklavec  wrote:
> 
> On 6 March 2016 at 18:01, Bradley Giesbrecht wrote:
>> 
>> We could remove all ports that depend on mysql4 and mysql5 exclusively or
>> via default variants. The remaining ports would then have their mysql4 or
>> mysql5 variant removed.
>> 
>> The removed ports could be brought back if someone was interested in fixing
>> them.
> 
> That would be a bit harsh, wouldn't it? I bet that a number of ports
> would work with a different database without any additional patches.
> 
> I asked long ago if someone could provide guidelines for transition,
> but nobody did. It's not clear to me what the suggested way is, so
> that we could at least make some consistent changes rather than every
> port switching to a different database with completely different code.

I propose that someone other then me fix base to allow variants with the same 
naming convention as port names. This would allow variants with hyphens and 
dots. So the hyphens will be problematic, ok.

But the dots are somewhat important. Is +mariadb_101 mariadb 1.01 10.1 or 100?

I looked at updating base to allow dots in variant names and it didn’t look to 
difficult, at one point a I had a patch that worked in my testing, but I didn’t 
feel comfortable enough with base to make the change and the discussion on the 
list didn’t get much traction although I remember people mentioning it would be 
good to not have ambiguous version strings in names of ports and variants.

Coming up with a standard variant name for the mariadb-10.0 and mariadb-10.1 is 
the place to start in my opinion.

A second thought is to create a mysql portgroup which sets some helper 
variables like $mysql_lib_path, $mysql_include_path and $mysql_config_path. The 
portgroup might also have a method to create mysql stub variants to enforce 
variant naming conventions with commented code blocks to show how to test if a 
variant is active and examples of using the portgroups helper variables.


Can we get dots in variant names into base soon?
If not, what should be the standard for variant names when the port it is 
targeting has dots in its name?

—
Brad

___
macports-dev mailing list
macports-dev@lists.macosforge.org
https://lists.macosforge.org/mailman/listinfo/macports-dev


Re: [146276] trunk/dports/databases

2016-03-06 Thread Mojca Miklavec
On 6 March 2016 at 18:01, Bradley Giesbrecht wrote:
>
> We could remove all ports that depend on mysql4 and mysql5 exclusively or
> via default variants. The remaining ports would then have their mysql4 or
> mysql5 variant removed.
>
> The removed ports could be brought back if someone was interested in fixing
> them.

That would be a bit harsh, wouldn't it? I bet that a number of ports
would work with a different database without any additional patches.

I asked long ago if someone could provide guidelines for transition,
but nobody did. It's not clear to me what the suggested way is, so
that we could at least make some consistent changes rather than every
port switching to a different database with completely different code.

Mojca
___
macports-dev mailing list
macports-dev@lists.macosforge.org
https://lists.macosforge.org/mailman/listinfo/macports-dev


Re: [146276] trunk/dports/databases

2016-03-06 Thread Bradley Giesbrecht
> On Mar 5, 2016, at 3:39 AM, Ryan Schmidt  wrote:
> 
>> 
>> On Mar 3, 2016, at 6:10 PM, Bradley Giesbrecht  wrote:
>> 
>>> On Mar 3, 2016, at 12:14 AM, Juan Manuel Palacios  wrote:
>>> 
>>> Bradley,
>>> 
>>> I think you and I already discussed this in some thread, but I can’t 
>>> remember right now… 
>>> 
>>> What’s the point of the mysql5 port now a-days? I’d say it’s redundant with 
>>> mysql51, but not even, ‘cause it’s outdated with respect to it, 5.1.72 Vs. 
>>> 5.1.73. Shouldn’t we just obsolete that port and have it replaced by 
>>> mysql51?
>> 
>> 
>> These ports have dependencies on mysql5 either directly or through variants:
>> port info --name --variants depends:"(\W|^)mysql5(\W|$)" or 
>> variant:"(\W|^)mysql5(\W|$)" | grep -E "^name:|^variants:.*mysql5|—"
>> 
>> Ticket discussing issue:
>> https://trac.macports.org/ticket/43431
> 
> It's been so long, I think it's probably find to make mysql5 replaced_by 
> mysql51, even if there are still ports depending on mysql5. The old php5- 
> ports were deleted awhile ago, even though there are still ancient ports 
> declaring dependencies on them.


We could remove all ports that depend on mysql4 and mysql5 exclusively or via 
default variants. The remaining ports would then have their mysql4 or mysql5 
variant removed.

The removed ports could be brought back if someone was interested in fixing 
them.


Regards,
Bradley Giesbrecht (pixilla)

___
macports-dev mailing list
macports-dev@lists.macosforge.org
https://lists.macosforge.org/mailman/listinfo/macports-dev


Re: [146276] trunk/dports/databases

2016-03-05 Thread Ryan Schmidt

> On Mar 3, 2016, at 6:10 PM, Bradley Giesbrecht  wrote:
> 
>> On Mar 3, 2016, at 12:14 AM, Juan Manuel Palacios  wrote:
>> 
>> Bradley,
>> 
>> I think you and I already discussed this in some thread, but I can’t 
>> remember right now… 
>> 
>> What’s the point of the mysql5 port now a-days? I’d say it’s redundant with 
>> mysql51, but not even, ‘cause it’s outdated with respect to it, 5.1.72 Vs. 
>> 5.1.73. Shouldn’t we just obsolete that port and have it replaced by mysql51?
> 
> 
> These ports have dependencies on mysql5 either directly or through variants:
> port info --name --variants depends:"(\W|^)mysql5(\W|$)" or 
> variant:"(\W|^)mysql5(\W|$)" | grep -E "^name:|^variants:.*mysql5|—"
> 
> Ticket discussing issue:
> https://trac.macports.org/ticket/43431

It's been so long, I think it's probably find to make mysql5 replaced_by 
mysql51, even if there are still ports depending on mysql5. The old php5- ports 
were deleted awhile ago, even though there are still ancient ports declaring 
dependencies on them.


___
macports-dev mailing list
macports-dev@lists.macosforge.org
https://lists.macosforge.org/mailman/listinfo/macports-dev


Re: [146276] trunk/dports/databases

2016-03-03 Thread Bradley Giesbrecht
> On Mar 3, 2016, at 12:14 AM, Juan Manuel Palacios  wrote:
> 
> Bradley,
> 
> I think you and I already discussed this in some thread, but I can’t remember 
> right now… 
> 
> What’s the point of the mysql5 port now a-days? I’d say it’s redundant with 
> mysql51, but not even, ‘cause it’s outdated with respect to it, 5.1.72 Vs. 
> 5.1.73. Shouldn’t we just obsolete that port and have it replaced by mysql51?


These ports have dependencies on mysql5 either directly or through variants:
port info --name --variants depends:"(\W|^)mysql5(\W|$)" or 
variant:"(\W|^)mysql5(\W|$)" | grep -E "^name:|^variants:.*mysql5|—"

Ticket discussing issue:
https://trac.macports.org/ticket/43431


Regards,
Bradley Giesbrecht (pixilla)


___
macports-dev mailing list
macports-dev@lists.macosforge.org
https://lists.macosforge.org/mailman/listinfo/macports-dev


Re: [146276] trunk/dports/databases

2016-03-03 Thread Juan Manuel Palacios
Bradley,

I think you and I already discussed this in some thread, but I can’t remember 
right now… 
> Modified: trunk/dports/databases/mysql5/Portfile (146275 => 146276)
> 
> --- trunk/dports/databases/mysql5/Portfile2016-03-03 06:16:58 UTC (rev 
> 146275)
> +++ trunk/dports/databases/mysql5/Portfile2016-03-03 07:25:10 UTC (rev 
> 146276)
> @@ -6,7 +6,7 @@
>  namemysql5
>  version 5.1.72
>  # Please set revision_client and revision_server to 0 if you bump version.
> -set revision_client 0
> +set revision_client 1
>  set revision_server 0
>  set version_branch  [join [lrange [split ${version} .] 0 1] .]
>  homepagehttp://www.mysql.com/ <http://www.mysql.com/>
>  <>Modified: trunk/dports/databases/mysql51/Portfile (146275 => 146276)
> 
> --- trunk/dports/databases/mysql51/Portfile   2016-03-03 06:16:58 UTC (rev 
> 146275)
> +++ trunk/dports/databases/mysql51/Portfile   2016-03-03 07:25:10 UTC (rev 
> 146276)
> @@ -7,7 +7,7 @@
>  set name_mysql  ${name}
>  version 5.1.73
>  # Set revision_client and revision_server to 0 if you bump version.
> -set revision_client 2
> +set revision_client 3
>  set revision_server 0
>  set version_branch  [join [lrange [split ${version} .] 0 1] .]
>  categories  databases
>  <>
What’s the point of the mysql5 port now a-days? I’d say it’s redundant with 
mysql51, but not even, ‘cause it’s outdated with respect to it, 5.1.72 Vs. 
5.1.73. Shouldn’t we just obsolete that port and have it replaced by mysql51?


- jmpp

___
macports-dev mailing list
macports-dev@lists.macosforge.org
https://lists.macosforge.org/mailman/listinfo/macports-dev