Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#14 | Add-Twig-to-COPYRIGHT.txt-and-copy-AUTHORS.patch | 3.17 KB | greg.1.anderson |
#13 | 0001-Add-Twig-to-COPYRIGHT.txt-and-copy-Twig-LICENSE-to-c.patch | 2.47 KB | greg.1.anderson |
#9 | 1591686.patch | 384.48 KB | RobLoach |
#8 | twig.patch | 384.19 KB | RobLoach |
twig.patch | 379.46 KB | chx | |
Comments
Comment #1
aspilicious CreditAttribution: aspilicious commentedI don't wonna be a dick but does twig has a signoff from everyone (including Dries?)
Comment #2
bojanz CreditAttribution: bojanz commentedAs 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.
Comment #3
chx CreditAttribution: chx commentedI 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.
Comment #4
Crell CreditAttribution: Crell commented+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.
Comment #5
catchThis is fine with me, but I've never actually seen Dries weigh in on Twig yet, so over to him.
Comment #6
chx CreditAttribution: chx commentedWriteups: http://groups.drupal.org/node/234793 http://groups.drupal.org/node/234838
Comment #7
jenlamptonFor reference: http://groups.drupal.org/node/234793 and http://groups.drupal.org/node/234838
edit: chx is faster than I am :/
Comment #8
RobLoachdrush composer create-project twig/twig
)Comment #9
RobLoachchx brought up a good point that we need to use PEAR-style autoloading for this, which is "Prefixes" in Symfony's autoloader.
Comment #10
chx CreditAttribution: chx commentedComment #11
Dries CreditAttribution: Dries commentedAlright, 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! :-)
Comment #12
Dries CreditAttribution: Dries commentedNo longer needs to assigned to me.
Comment #13
greg.1.anderson CreditAttribution: greg.1.anderson commentedTo 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.
Comment #14
greg.1.anderson CreditAttribution: greg.1.anderson commentedWe should probably also have AUTHORS, since LICENSE references it.
Comment #15
chx CreditAttribution: chx commentedThanks!
Comment #16
jhodgdonI committed the patch in #14 to 8.x.
Comment #17
chx CreditAttribution: chx commentedTitling back and thanks for the emergency commit.
Comment #18
RobLoachThanks Greg! Not sure where those went in my patch :-) .
Comment #19.0
podarokUpdated issue summary.