Part of meta-issue #1788918: [GTD] [META] Prepare for Twig main engine core inclusion until December 1

We will see how we will integrate with it. But first, let's add it so that the integration patches are reviewable.

Files: 
CommentFileSizeAuthor
#14 Add-Twig-to-COPYRIGHT.txt-and-copy-AUTHORS.patch3.17 KBgreg.1.anderson
PASSED: [[SimpleTest]]: [MySQL] 36,891 pass(es).
[ View ]
#13 0001-Add-Twig-to-COPYRIGHT.txt-and-copy-Twig-LICENSE-to-c.patch2.47 KBgreg.1.anderson
PASSED: [[SimpleTest]]: [MySQL] 36,901 pass(es).
[ View ]
#9 1591686.patch384.48 KBRobLoach
PASSED: [[SimpleTest]]: [MySQL] 36,857 pass(es).
[ View ]
#8 twig.patch384.19 KBRobLoach
PASSED: [[SimpleTest]]: [MySQL] 36,859 pass(es).
[ View ]
twig.patch379.46 KBchx
PASSED: [[SimpleTest]]: [MySQL] 36,597 pass(es).
[ View ]

Comments

aspilicious’s picture

I don't wonna be a dick but does twig has a signoff from everyone (including Dries?)

bojanz’s picture

As much as the Symfony stuff did when it was committed.

That said, committing huge chunks of external code like this makes me sad -> we need Composer.

chx’s picture

I have yet to find any dissent in #1499460: [meta] New theme system and yeah much like Symfony if it is not working out then we can rip it out but I really would prefer not to submit 300K+ patches when we actually try to use this.

Crell’s picture

+1 on this issue for now.

+1 for following up with #1424924: Use Composer for updating Symfony components (without removing Symfony code from repo)

+1 for then switching to pure-composer later.

catch’s picture

Assigned:Unassigned» Dries

This is fine with me, but I've never actually seen Dries weigh in on Twig yet, so over to him.

chx’s picture

jenlampton’s picture

For reference: http://groups.drupal.org/node/234793 and http://groups.drupal.org/node/234838

edit: chx is faster than I am :/

RobLoach’s picture

Status:Reviewed & tested by the community» Needs review
StatusFileSize
new384.19 KB
PASSED: [[SimpleTest]]: [MySQL] 36,859 pass(es).
[ View ]
  • Updated to dev-master (drush composer create-project twig/twig)
  • Added Twig to our PSR-0 class-loader
RobLoach’s picture

StatusFileSize
new384.48 KB
PASSED: [[SimpleTest]]: [MySQL] 36,857 pass(es).
[ View ]

chx brought up a good point that we need to use PEAR-style autoloading for this, which is "Prefixes" in Symfony's autoloader.

chx’s picture

Status:Needs review» Reviewed & tested by the community
Dries’s picture

Status:Reviewed & tested by the community» Fixed

Alright, folks. I committed this to 8.x to encourage all of us to make progress on this initiative. If the theme system revamp doesn't come to fruition, we will roll this back. Looking forward to the "real" follow-up patches. Ready, set, go! :-)

Dries’s picture

Assigned:Dries» Unassigned

No longer needs to assigned to me.

greg.1.anderson’s picture

Title:Add Twig itself» Twig copyright missing from COPYRIGHT.txt
Status:Fixed» Needs review
StatusFileSize
new2.47 KB
PASSED: [[SimpleTest]]: [MySQL] 36,901 pass(es).
[ View ]

To comply with core/vendor/README.txt, COPYRIGHT.txt should contain the copyright to twig. To comply with the Twig license, the file LICENSE should also be included in the core/vendor/Twig directory.

greg.1.anderson’s picture

StatusFileSize
new3.17 KB
PASSED: [[SimpleTest]]: [MySQL] 36,891 pass(es).
[ View ]

We should probably also have AUTHORS, since LICENSE references it.

chx’s picture

Status:Needs review» Reviewed & tested by the community

Thanks!

jhodgdon’s picture

Status:Reviewed & tested by the community» Fixed

I committed the patch in #14 to 8.x.

chx’s picture

Title:Twig copyright missing from COPYRIGHT.txt» Add Twig itself

Titling back and thanks for the emergency commit.

RobLoach’s picture

Thanks Greg! Not sure where those went in my patch :-) .

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

podarok’s picture

Issue summary:View changes

Updated issue summary.